Message ID | 20171027114716.29198-1-peter@korsgaard.com |
---|---|
State | Accepted |
Headers | show |
Series | lvm2: make basic package available under musl | expand |
Hello, On Fri, 27 Oct 2017 13:47:16 +0200, Peter Korsgaard wrote: > config BR2_PACKAGE_LVM2_STANDARD_INSTALL > bool "standard install instead of only dmsetup" > + # http://lists.busybox.net/pipermail/buildroot/2016-August/170592.html > + depends on !BR2_TOOLCHAIN_USES_MUSL > default y if !BR2_PACKAGE_LVM2_DMSETUP_ONLY # legacy 2013.11 > help > Install the standard suite of lvm2 programs. When this option > @@ -34,10 +34,15 @@ config BR2_PACKAGE_LVM2_STANDARD_INSTALL > > config BR2_PACKAGE_LVM2_APP_LIBRARY > bool "install application library" > + # http://lists.busybox.net/pipermail/buildroot/2016-August/170592.html > + depends on !BR2_TOOLCHAIN_USES_MUSL > select BR2_PACKAGE_LVM2_STANDARD_INSTALL > help > Install application library (liblvm2app). > > +comment "lvm2 standard install needs a glibc or uClibc toolchain" > + depends on BR2_TOOLCHAIN_USES_MUSL For consistency, I've moved this comment below the "standard install" option, and added another comment for the "application library" option. > + > endif > > comment "lvm2 needs a glibc or uClibc toolchain w/ threads, dynamic library" You had forgotten to update this comment: lvm2 no longer needs glibc or uClibc, and you also had forgotten to drop the BR2_TOOLCHAIN_USES_MUSL part of this comment dependencies. Applied with those nits fixed. Thanks! Thomas
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes: Hi, >> Install application library (liblvm2app). >> >> +comment "lvm2 standard install needs a glibc or uClibc toolchain" >> + depends on BR2_TOOLCHAIN_USES_MUSL > For consistency, I've moved this comment below the "standard install" > option, and added another comment for the "application library" option. Ok. In the general case this would screw up indentation of further sub option, but as the dependencies are the same this won't happen here. >> comment "lvm2 needs a glibc or uClibc toolchain w/ threads, dynamic library" > You had forgotten to update this comment: lvm2 no longer needs glibc or > uClibc, and you also had forgotten to drop the BR2_TOOLCHAIN_USES_MUSL > part of this comment dependencies. Ups, thanks for fixing it!
diff --git a/package/cryptsetup/Config.in b/package/cryptsetup/Config.in index 5cd4676eaa..d24f7b0af6 100644 --- a/package/cryptsetup/Config.in +++ b/package/cryptsetup/Config.in @@ -3,7 +3,6 @@ config BR2_PACKAGE_CRYPTSETUP depends on BR2_TOOLCHAIN_HAS_THREADS # lvm2 depends on BR2_USE_MMU # lvm2 depends on !BR2_STATIC_LIBS # lvm2 - depends on !BR2_TOOLCHAIN_USES_MUSL # lvm2 select BR2_PACKAGE_POPT select BR2_PACKAGE_LVM2 select BR2_PACKAGE_UTIL_LINUX @@ -14,7 +13,6 @@ config BR2_PACKAGE_CRYPTSETUP https://gitlab.com/cryptsetup/cryptsetup -comment "cryptsetup needs a glibc or uClibc toolchain w/ threads, dynamic library" +comment "cryptsetup needs a toolchain w/ threads, dynamic library" depends on BR2_USE_MMU - depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS \ - || BR2_TOOLCHAIN_USES_MUSL + depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS diff --git a/package/dmraid/Config.in b/package/dmraid/Config.in index d0acc4fd29..aa98c853a1 100644 --- a/package/dmraid/Config.in +++ b/package/dmraid/Config.in @@ -3,7 +3,6 @@ config BR2_PACKAGE_DMRAID depends on BR2_TOOLCHAIN_HAS_THREADS # lvm2 depends on BR2_USE_MMU # lvm2 depends on !BR2_STATIC_LIBS # lvm2 - depends on !BR2_TOOLCHAIN_USES_MUSL # lvm2 select BR2_PACKAGE_LVM2 help dmraid discovers, activates, deactivates and displays @@ -13,7 +12,6 @@ config BR2_PACKAGE_DMRAID dmraid uses the Linux device-mapper to create devices with respective mappings for the ATARAID sets discovered. -comment "dmraid needs a glibc or uClibc toolchain w/ threads, dynamic library" +comment "dmraid needs a toolchain w/ threads, dynamic library" depends on BR2_USE_MMU - depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS \ - || BR2_TOOLCHAIN_USES_MUSL + depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS diff --git a/package/lvm2/Config.in b/package/lvm2/Config.in index 32c763978f..7ea4d3e260 100644 --- a/package/lvm2/Config.in +++ b/package/lvm2/Config.in @@ -3,8 +3,6 @@ config BR2_PACKAGE_LVM2 depends on BR2_TOOLCHAIN_HAS_THREADS depends on BR2_USE_MMU # needs fork() depends on !BR2_STATIC_LIBS # It fails to build statically - # http://lists.busybox.net/pipermail/buildroot/2016-August/170592.html - depends on !BR2_TOOLCHAIN_USES_MUSL help This is LVM2, the rewrite of The Linux Logical Volume Manager. LVM supports enterprise level volume management of disk and @@ -27,6 +25,8 @@ if BR2_PACKAGE_LVM2 config BR2_PACKAGE_LVM2_STANDARD_INSTALL bool "standard install instead of only dmsetup" + # http://lists.busybox.net/pipermail/buildroot/2016-August/170592.html + depends on !BR2_TOOLCHAIN_USES_MUSL default y if !BR2_PACKAGE_LVM2_DMSETUP_ONLY # legacy 2013.11 help Install the standard suite of lvm2 programs. When this option @@ -34,10 +34,15 @@ config BR2_PACKAGE_LVM2_STANDARD_INSTALL config BR2_PACKAGE_LVM2_APP_LIBRARY bool "install application library" + # http://lists.busybox.net/pipermail/buildroot/2016-August/170592.html + depends on !BR2_TOOLCHAIN_USES_MUSL select BR2_PACKAGE_LVM2_STANDARD_INSTALL help Install application library (liblvm2app). +comment "lvm2 standard install needs a glibc or uClibc toolchain" + depends on BR2_TOOLCHAIN_USES_MUSL + endif comment "lvm2 needs a glibc or uClibc toolchain w/ threads, dynamic library"
The basic lvm2 package (libdevmapper / dmsetup) does build under musl, only the standard (full) installation doesn't. Many setups only need the basic package, so move the !musl dependencies down to the sub options and adjust the reverse dependencies (cryptsetup/dmraid) to match. Signed-off-by: Peter Korsgaard <peter@korsgaard.com> --- package/cryptsetup/Config.in | 6 ++---- package/dmraid/Config.in | 6 ++---- package/lvm2/Config.in | 9 +++++++-- 3 files changed, 11 insertions(+), 10 deletions(-)