Re: [PATCH 01/15] Support multiarch under OS directory

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Thu, Jan 3, 2013 at 7:20 AM, Andreas Jaeger <aj@suse.com> wrote:
> On 12/23/2012 04:02 AM, Andi Kleen wrote:
>>
>> From: "H.J. Lu" <hjl.tools@gmail.com>
>
>
> Let's push this patch interdependent of the others in - but it needs a
> proper reasoning what it will do and why it's needed.
>
> Please explain what you want to achieve here,

configure was supposed to support multiarch directory
under sysdeps/unix/sysv/linux/x86_64, as m0sub and msub
used in directory scan:

try="${d}sysdeps$m0$m0sub$b$v$o$m$msub$a"

But we never set m0sub and msub.  As the result, we never
check multiarch directory under OS directory.  My patch adds
the missing m0sub and msub. Ignoring white spaces, my patch is:

diff --git a/configure b/configure
index 8799b7d..bf73d05 100755
--- a/configure
+++ b/configure
@@ -4347,11 +4347,13 @@ sysnames_add_ons=
 sysnames=
 for b in $base ''; do
   for m0 in $mach ''; do
+    for m0sub in $mach ''; do
       for v in /$vendor ''; do
   test "$v" = / && continue
   for o in /$ostry ''; do
     test "$o" = / && continue
     for m in $multi_arch_d $mach ''; do
+      for msub in $multi_arch_d $mach ''; do
         for d in $add_ons_pfx ''; do
      for a in $add_ons_sfx ''; do
        if test -n "$m0$m0sub$b$v$o$m$msub"; then
@@ -4384,6 +4386,8 @@ for b in $base ''; do
   done
       done
     done
+  done
+done

It is a correctness issue.

-- 
H.J.