[Buildroot] [PATCH 03/51] package/gcc: align gcc-final configure-cmds to the generic ones

Luca Ceresoli luca at lucaceresoli.net
Thu Dec 3 17:10:18 UTC 2015


Dear Yann,

Yann E. MORIN wrote:
> Since 7d6c79 (Compile static versions of gcc libraries) was applied, the
> generic configure commands have been updated, but those changes have not
> been propagated to the gcc-final custom configure commands.
>
> Update the gcc-final custom configure commands to better match the
> generic ones.
>
> We do not propagate --disable-dependency-tracking because it breaks the
> build, and --enable-shared (because of 169141a).
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> ---
>   package/gcc/gcc-final/gcc-final.mk | 25 ++++++++++++++++++++-----
>   1 file changed, 20 insertions(+), 5 deletions(-)
>
> diff --git a/package/gcc/gcc-final/gcc-final.mk b/package/gcc/gcc-final/gcc-final.mk
> index dbdb615..53199c3 100644
> --- a/package/gcc/gcc-final/gcc-final.mk
> +++ b/package/gcc/gcc-final/gcc-final.mk
> @@ -36,16 +36,31 @@ HOST_GCC_FINAL_PRE_CONFIGURE_HOOKS += HOST_GCC_CONFIGURE_SYMLINK
>   #
>   # So we must completely override the generic commands and provide our own.
>   #
> +# Except for --disable-dependency-tracking which breaks the build, and
> +# --enable-shared (see 169141a, gcc-final: disable shared build for
> +# static), we reproduce the exact same command as the autotools infra
> +# uses.
> +#
>   define  HOST_GCC_FINAL_CONFIGURE_CMDS
>   	(cd $(HOST_GCC_FINAL_SRCDIR) && rm -rf config.cache; \
> -		$(HOST_CONFIGURE_OPTS) \
> -		CFLAGS="$(HOST_CFLAGS)" \
> -		LDFLAGS="$(HOST_LDFLAGS)" \
> -		$(HOST_GCC_FINAL_CONF_ENV) \
> -		./configure \
> +	$(HOST_CONFIGURE_OPTS) \
> +	CFLAGS="$(HOST_CFLAGS)" \
> +	LDFLAGS="$(HOST_LDFLAGS)" \
> +	$(HOST_GCC_FINAL_CONF_ENV) \
> +	CONFIG_SITE=/dev/null \

With the indentation cleanup, the CONFIG_SITE addition might be
unnoticed (it did to me, at first). I suggest you mention its addition
in the commit message and reference the originating commit:

------------8<--------------
CONFIG_SITE from commit f8d4fe3 ("package/pkg-autotools: don't use
config.site when cross-compiling").
------------8<--------------

With that added (and even without):

Reviewed-by: Luca Ceresoli <luca at lucaceresoli.net>
[Tested building qemu_arm_versatile_defconfig]
Tested-by: Luca Ceresoli <luca at lucaceresoli.net>

-- 
Luca


More information about the buildroot mailing list