Re: [PATCH] shell: Don't include config.h for native helpers

[Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>]

On Tue, May 22, 2018 at 06:57:07PM +0300, Baruch Siach wrote:
> From: Peter Korsgaard <peter@xxxxxxxxxxxxx>
> 
> config.h contains settings for the cross compiler (most importantly
> 32/64bit versions of functions), so don't include it when calling the
> native compiler to build the helpers.
> 
> Otherwise we get build errors like:
> 
> /usr/bin/gcc -include ../config.h -DBSD=1 -DSHELL -DIFS_BROKEN  -g -O2 -Wall    -o mkinit mkinit.c
> In file included from /usr/include/sys/stat.h:107,
>                  from /usr/include/fcntl.h:38,
>                  from mkinit.c:50:
> /usr/include/bits/stat.h:117: error: redefinition of ‘struct stat’
> In file included from /usr/include/fcntl.h:38,
>                  from mkinit.c:50:
> /usr/include/sys/stat.h:504: error: redefinition of ‘stat’
> /usr/include/sys/stat.h:455: note: previous definition of ‘stat’ was here
> 
> Signed-off-by: Peter Korsgaard <peter@xxxxxxxxxxxxx>
> [baruch: apply to Makefile.am; update Peter's email address]
> Signed-off-by: Baruch Siach <baruch@xxxxxxxxxx>

Patch applied.  Thanks.
-- 
Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt