[Toybox] [PATCH] Clean warnings generated by scripts/genconfig.sh

[Rob Landley]

On 02/28/2012 01:16 AM, Georgi Chorbadzhiyski wrote:
> It looks like configure should not do CC="${CROSS_COMPILE}${CC}"
> unconditionally.
> Isn't it better to use ${CROSS_COMPILE}${CC} instead in the scripts
> instead of
> redefining CC. Something like this:
> 
> diff -r 763d581badae configure
> --- a/configure Sun Feb 26 22:04:37 2012 -0600
> +++ b/configure Tue Feb 28 09:15:14 2012 +0200
> @@ -9,6 +9,5 @@
>  CFLAGS="$CFLAGS -funsigned-char"
>  [ -z "$OPTIMIZE" ] && OPTIMIZE="-Os -ffunction-sections -fdata-sections
> -Wl,--gc-sections"
>  [ -z "$CC" ] && CC="cc"
> -CC="${CROSS_COMPILE}${CC}"
>  [ -z "$STRIP" ] && STRIP="${CROSS_COMPILE}strip"
>  [ -z "$HOSTCC" ] && HOSTCC=gcc
> diff -r 763d581badae scripts/genconfig.sh
> --- a/scripts/genconfig.sh      Sun Feb 26 22:04:37 2012 -0600
> +++ b/scripts/genconfig.sh      Tue Feb 28 09:15:14 2012 +0200
> @@ -13,7 +13,7 @@
>    # Probe for container support on target
> 
>    echo -e "# container support\nconfig TOYBOX_CONTAINER\n\tbool" ||
> return 1
> -  $CC -c -xc -o /dev/null - << EOF
> +  ${CROSS_COMPILE}${CC} -c -xc -o /dev/null - << EOF
>      #include <sched.h>
>      int x=CLONE_NEWNS|CLONE_NEWUTS|CLONE_NEWIPC|CLONE_NEWNET;
>  EOF
> diff -r 763d581badae scripts/make.sh
> --- a/scripts/make.sh   Sun Feb 26 22:04:37 2012 -0600
> +++ b/scripts/make.sh   Tue Feb 28 09:15:14 2012 +0200
> @@ -102,7 +102,7 @@
>    "$@"
>  }
> 
> -do_loudly $CC $CFLAGS -I . -o toybox_unstripped $OPTIMIZE main.c lib/*.c \
> +do_loudly ${CROSS_COMPILE}${CC} $CFLAGS -I . -o toybox_unstripped
> $OPTIMIZE main.c lib/*.c \
>    $TOYFILES -Wl,--as-needed,-lutil,--no-as-needed || exit 1
>  do_loudly $STRIP toybox_unstripped -o toybox || exit 1
>  # gcc 4.4's strip command is buggy, and doesn't set the executable bit on
> 

You're right.  I've been holding off on most of your patches until after
this release, but this one's needed now.

Unfortunately, saving your email message to a file and trying to "patch
-p1" it gave me all rejects, I think it's whitespace damaged or
something.  I'll apply the changes by hand, just FYI.

Thanks,

Rob