From mboxrd@z Thu Jan  1 00:00:00 1970
From: Yann E. MORIN <yann.morin.1998@free.fr>
Date: Tue, 8 Nov 2016 18:29:12 +0100
Subject: [Buildroot] [PATCH 2/6] package/pseudo: enforce the host bitness
In-Reply-To: <6bef7da4-a2b1-108f-286a-a395305b4d9c@mind.be>
References: <cover.1478553802.git.yann.morin.1998@free.fr>
 <74e4be0c7c6aa89e0ce33144e593d65d13ea6749.1478553802.git.yann.morin.1998@free.fr>
 <6bef7da4-a2b1-108f-286a-a395305b4d9c@mind.be>
Message-ID: <20161108172912.GC3554@free.fr>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

Arnout, All,

On 2016-11-07 23:16 +0100, Arnout Vandecappelle spake thusly:
> 
> 
> On 07-11-16 22:23, Yann E. MORIN wrote:
> > pseudo can detect the host bitness, but is not sure about it: it checks
> > what type of file /bin/sh is, using file(1).
> > 
> > However, in some conditions, /bin/sh can be of a different bitness than
> > the rest of the system (weird, but not impossible), which causes build
> > issues.
> > 
> > Just enforce the bitness, so that pseudo needs not (wrongly) guess it.
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Ga?l PORTAY <gael.portay@savoirfairelinux.com>
> > Cc: Patrick Keroulas <patrick.keroulas@savoirfairelinux.com>
> > Cc: Erico Nunes <nunes.erico@gmail.com>
> > Cc: Julien BOIBESSOT <julien.boibessot@free.fr>
> > ---
> >  package/pseudo/pseudo.mk | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/package/pseudo/pseudo.mk b/package/pseudo/pseudo.mk
> > index 67538a6..3b0ba86 100644
> > --- a/package/pseudo/pseudo.mk
> > +++ b/package/pseudo/pseudo.mk
> > @@ -17,6 +17,7 @@ HOST_PSEUDO_DEPENDENCIES = host-attr host-sqlite
> >  # configure script is not generated by autoconf, so passing --libdir
> >  # is necessary, even if the infrastructure passes --prefix already.
> >  HOST_PSEUDO_CONF_OPTS = \
> > +	--bits=$(HOSTARCH_BITS) \
> 
>  I would be just as happy with
> 	--bits=$(if $(filter %64,$(HOSTARCH)),32,64)

Yep, that's nice. Except it's reversed, isn't it? I.e. if we find '64'
in HOSTARCH, then we want to use '64', not '32', right? So:

    --bits=$(if $(filter %64,$(HOSTARCH)),64,32)

(Unless I missed something...)

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'