[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