Re: [Buildroot] [PATCH] package/gcc: Disable uclibc for powerpc64 and powerpc64le

[Arnout Vandecappelle <arnout@mind.be>]

On 05/05/2022 12:09, Joel Stanley wrote:
> uClibc-ng does not (and has never) supported the 64 bit powerpc
> architecture.

  Exactly, that's why we have

config BR2_TOOLCHAIN_BUILDROOT_UCLIBC
	bool "uClibc-ng"
	depends on BR2_aarch64 || BR2_aarch64_be || BR2_arcle   || BR2_arceb  || \
		   BR2_arm     || BR2_armeb    || \
		   BR2_i386    || BR2_m68k   || BR2_microblaze || \
		   BR2_mips    || BR2_mipsel || BR2_mips64 || BR2_mips64el || \
		   BR2_or1k    || BR2_powerpc || BR2_RISCV_64 || \
		   BR2_sh4     || BR2_sh4eb   || BR2_sparc || BR2_xtensa || \
		   BR2_x86_64

  Can you give a concrete defconfig where it is possible to select uClibc for 
powerpc64?

  Assuming this is a mistake, I've marked as Rejected.

> 
> Signed-off-by: Joel Stanley <joel@jms.id.au>
> ---
> I'm a bit confused as to why uclibc was ever an option. Perhaps I missed
> something! I did a search of the mailing list archive and the best I
> found was this post from Thomas:
> 
>   https://lore.kernel.org/buildroot/20160817193721.0cc1466d@free-electrons.com/
> 
> Signed-off-by: Joel Stanley <joel@jms.id.au>
> ---
>   package/gcc/Config.in.host | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/package/gcc/Config.in.host b/package/gcc/Config.in.host
> index a1fe192d3a24..3364860ec7a4 100644
> --- a/package/gcc/Config.in.host
> +++ b/package/gcc/Config.in.host
> @@ -31,6 +31,8 @@ config BR2_GCC_VERSION_9_X
>   	# upstream gcc. C-SKY gcc upstream support not tested
>   	# with upstream binutils and glibc.
>   	depends on !BR2_csky
> +	# 64 bit powerpc is not supported by uclibc
> +	depends on !(BR2_TOOLCHAIN_USES_UCLIBC && (BR2_powerpc64 || BR2_powerpc64le))

  This isn't the way to do that - the result would be that if you have uClibc 
selected, there is no choice option available for GCC. You have to do it in the 
uClibc config (like it is now).

  Regards,
  Arnout

>   	select BR2_TOOLCHAIN_GCC_AT_LEAST_9
>   
>   config BR2_GCC_VERSION_10_X
> @@ -43,6 +45,8 @@ config BR2_GCC_VERSION_10_X
>   	# upstream gcc. C-SKY gcc upstream support not tested
>   	# with upstream binutils and glibc.
>   	depends on !BR2_csky
> +	# 64 bit powerpc is not supported by uclibc
> +	depends on !(BR2_TOOLCHAIN_USES_UCLIBC && (BR2_powerpc64 || BR2_powerpc64le))
>   	select BR2_TOOLCHAIN_GCC_AT_LEAST_10
>   
>   config BR2_GCC_VERSION_11_X
> @@ -54,6 +58,8 @@ config BR2_GCC_VERSION_11_X
>   	# that need to be reverted since gcc 8.4, 9.3 and 10.1.
>   	# https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98784
>   	depends on !BR2_sparc
> +	# 64 bit powerpc is not supported by uclibc
> +	depends on !(BR2_TOOLCHAIN_USES_UCLIBC && (BR2_powerpc64 || BR2_powerpc64le))
>   	select BR2_TOOLCHAIN_GCC_AT_LEAST_11
>   
>   endchoice
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot