Message ID | AS1P250MB060843CF993EF8CB658A1142A9F82@AS1P250MB0608.EURP250.PROD.OUTLOOK.COM |
---|---|
State | Superseded |
Headers | show |
Series | package/libzenoh-c: add GLIBC and aarch64, x86_64 and arm dependencies | expand |
Michel, All, On 2024-06-04 07:43 +0000, Michel Alex spake thusly: > zenoh-c only have support for following targets: > aarch64-unknown-linux-gnu, x86_64-unknown-linux-gnu, arm-unknown-linux-gnueabi [--SNIP--] > diff --git a/package/libzenoh-c/Config.in b/package/libzenoh-c/Config.in > index d22807c047..06e56c0879 100644 > --- a/package/libzenoh-c/Config.in > +++ b/package/libzenoh-c/Config.in > @@ -1,6 +1,17 @@ > +config BR2_PACKAGE_LIBZENOH_C_ARCH_SUPPORTS > + bool > + default y if BR2_aarch64 > + default y if BR2_arm > + default y if BR2_x86_64 > + depends on BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS > + depends on BR2_TOOLCHAIN_USES_GLIBC The dependency on the C library should not be in the _ARCH_SUPPORTS symbol. Indeed, the architecture is something the user can't changem it's what it is, but the C library is software, so the user has the option to choose. So, when the architecture is not supported, there is no reason to show the main symbol, and there is no reason to show the comment either. But when the architecture is supported, we want to tell the user the have the option of changing the C library to use that package. That's why in my quickly shipped code snippet in my previous mail, the dependency on the C library was not added to _ARCH_SUPPORTS, but to the main symbol and, as an inverted test, to the comment. Can you fix and respin, please? Regards, Yann E. MORIN. > +comment "libzenoh-c needs a glibc toolchain and supports only aarch64, x86_64 and arm architectures" > + depends on !BR2_PACKAGE_LIBZENOH_C_ARCH_SUPPORTS > + > config BR2_PACKAGE_LIBZENOH_C > bool "libzenoh-c" > - depends on BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS > + depends on BR2_PACKAGE_LIBZENOH_C_ARCH_SUPPORTS > select BR2_PACKAGE_HOST_RUSTC > help > This package provides a C binding based on the main > -- > 2.34.1 > > _______________________________________________ > buildroot mailing list > buildroot@buildroot.org > https://lists.buildroot.org/mailman/listinfo/buildroot
diff --git a/package/libzenoh-c/Config.in b/package/libzenoh-c/Config.in index d22807c047..06e56c0879 100644 --- a/package/libzenoh-c/Config.in +++ b/package/libzenoh-c/Config.in @@ -1,6 +1,17 @@ +config BR2_PACKAGE_LIBZENOH_C_ARCH_SUPPORTS + bool + default y if BR2_aarch64 + default y if BR2_arm + default y if BR2_x86_64 + depends on BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS + depends on BR2_TOOLCHAIN_USES_GLIBC + +comment "libzenoh-c needs a glibc toolchain and supports only aarch64, x86_64 and arm architectures" + depends on !BR2_PACKAGE_LIBZENOH_C_ARCH_SUPPORTS + config BR2_PACKAGE_LIBZENOH_C bool "libzenoh-c" - depends on BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS + depends on BR2_PACKAGE_LIBZENOH_C_ARCH_SUPPORTS select BR2_PACKAGE_HOST_RUSTC help This package provides a C binding based on the main
The manual of zenoh-c says that: zenoh-c only have support for following targets: aarch64-unknown-linux-gnu, x86_64-unknown-linux-gnu, arm-unknown-linux-gnueabi This patch provides a blind option BR2_PACKAGE_LIBZENOH_C_ARCH_SUPPORTS, which is default y for above architectures. Additionally adds a dependency on GLIBC because one of the autobuilders had a problem with MUSL. Moved also the BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS from package's dependency list to the BR2_PACKAGE_LIBZENOH_C_ARCH_SUPPORTS option. Fixes: http://autobuild.buildroot.net/results/680f25769526e5eef43d254a10a525b165846c75 http://autobuild.buildroot.net/results/31ed69cb2ffe8e9bc13a27c3fda16c27e9855b56 http://autobuild.buildroot.net/results/fdc74b8a42388b51cc4197c38b552559b97cbb8c http://autobuild.buildroot.net/results/6e7ad53d44b417f6aa62acd8b2396720f5a14e82 http://autobuild.buildroot.net/results/c85927b1dd25a84981adc5cc537eb07d02da40bc http://autobuild.buildroot.net/results/5389ac0174791b3d899fda7769b4f0f2dfeb805b http://autobuild.buildroot.net/results/285bd8f5aa338dbaf9ac0e5f78593b10bd54cbb5 Signed-off-by: Alex Michel <alex.michel@wiedemann-group.com> --- package/libzenoh-c/Config.in | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-)