Re: [Buildroot] [PATCH 0/7] Fix a number of toolchain dependency issues

["Yann E. MORIN" <yann.morin.1998@free.fr>]

James, Thomas, All,

On 2022-06-05 21:42 +0200, Thomas Petazzoni spake thusly:
> The starting point for this series is the investigation of several
> build failures affecting the master branch.
> 
> Most of those failures are due to the fact that we can currently
> create configurations for which no C library is available. This
> happens on architectures where glibc is the only supported C library,
> but BR2_STATIC_LIBS is enabled.
> 
> This is taken care of by the first four patches, which are basically
> split from a single patch posted by James Hilliard.
> 
> Then, another problem is that a recent commit allowed glibc only for
> powerpc8 on the ppc64le architecture, leaving a number of other
> PowerPC variants selectable with ppc64le, but for which no C library
> was available.
> 
> Finally, the toolchain-external-booltin package had a similar problem
> to the glibc/BR2_STATIC_LIBS issue: the package would be visible even
> if BR2_STATIC_LIBS=y and the selected CPU architecture is only
> supported by glibc.
> 
> Thomas
> 
> James Hilliard (4):
>   package/uclibc: introduce and use BR2_PACKAGE_UCLIBC_ARCH_SUPPORTS and
>     BR2_PACKAGE_UCLIBC_SUPPORTS
>   package/glibc: introduce and use BR2_PACKAGE_GLIBC_ARCH_SUPPORTS and
>     BR2_PACKAGE_GLIBC_SUPPORTS
>   package/musl: introduce and use BR2_PACKAGE_MUSL_ARCH_SUPPORTS and
>     BR2_PACKAGE_MUSL_SUPPORTS

Those three patches, applied to master, thanks.

I've however re-ordered them so that glibc comes first, which is then
used as an argument for keeping the same construct in uClibc-ng (so I
tweaked the commit message slightly), and to make the same argument in
musl as well (where I duplicated the commit log from uClibc-ng).

Regards,
Yann E. MORIN.

>   Config.in: only allow BR2_STATIC_LIBS on supported libc/arch
> 
> Thomas Petazzoni (3):
>   arch/Config.in.powerpc: disable some variants on ppc64le
>   support/scripts/gen-bootlin-toolchains: properly take into account
>     !BR2_STATIC_LIBS for glibc toolchains
>   toolchain/toolchain-external/toolchain-external-bootlin: regenerate
>     with correct !BR2_STATIC_LIBS handling
> 
>  Config.in                                     |  2 +
>  arch/Config.in.powerpc                        | 10 +++++
>  package/glibc/Config.in                       | 38 ++++++++++++++++
>  package/musl/Config.in                        | 28 ++++++++++++
>  package/uclibc/Config.in                      | 28 ++++++++++++
>  support/scripts/gen-bootlin-toolchains        |  5 ++-
>  toolchain/toolchain-buildroot/Config.in       | 43 ++++---------------
>  .../Config.in.options                         | 10 ++---
>  8 files changed, 124 insertions(+), 40 deletions(-)
> 
> -- 
> 2.35.3
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot