diff mbox series

util-linux: add missing dynamic library dependency

Message ID c446f43f2cc6b98efd08a0e499bb192337e26271.1505798272.git.baruch@tkos.co.il
State Accepted
Headers show
Series util-linux: add missing dynamic library dependency | expand

Commit Message

Baruch Siach Sept. 19, 2017, 5:17 a.m. UTC
Commit f52af619185 (util-linux: add menu items for login utilities)
forgot to propagate the !BR2_STATIC_LIBS dependency of linux-pam to
BR2_PACKAGE_UTIL_LINUX_LOGIN. Fix this.

Fixes:
http://autobuild.buildroot.net/results/e47/e472891a656e40f33081d9a5db1b1b6d71c1389f/
http://autobuild.buildroot.net/results/9e4/9e443720ca247292ea612851962c611a9bcd3450/
http://autobuild.buildroot.net/results/88f/88f4cecd7e2844701b0e75279c112e37283fc7f9/

Cc: Carlos Santos <casantos@datacom.ind.br>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 package/util-linux/Config.in | 1 +
 1 file changed, 1 insertion(+)

Comments

Carlos Santos Sept. 19, 2017, 11:06 a.m. UTC | #1
> De: "Baruch Siach" <baruch@tkos.co.il>
> Para: buildroot@busybox.net
> Cc: "Baruch Siach" <baruch@tkos.co.il>, "Carlos Santos" <casantos@datacom.ind.br>
> Enviadas: Terça-feira, 19 de setembro de 2017 2:17:52
> Assunto: [PATCH] util-linux: add missing dynamic library dependency

> Commit f52af619185 (util-linux: add menu items for login utilities)
> forgot to propagate the !BR2_STATIC_LIBS dependency of linux-pam to
> BR2_PACKAGE_UTIL_LINUX_LOGIN. Fix this.
> 
> Fixes:
> http://autobuild.buildroot.net/results/e47/e472891a656e40f33081d9a5db1b1b6d71c1389f/
> http://autobuild.buildroot.net/results/9e4/9e443720ca247292ea612851962c611a9bcd3450/
> http://autobuild.buildroot.net/results/88f/88f4cecd7e2844701b0e75279c112e37283fc7f9/
> 
> Cc: Carlos Santos <casantos@datacom.ind.br>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
> package/util-linux/Config.in | 1 +
> 1 file changed, 1 insertion(+)
> 
> diff --git a/package/util-linux/Config.in b/package/util-linux/Config.in
> index 80aab881a0da..5a600c333db3 100644
> --- a/package/util-linux/Config.in
> +++ b/package/util-linux/Config.in
> @@ -173,6 +173,7 @@ config BR2_PACKAGE_UTIL_LINUX_LOGIN
> 	bool "login"
> 	depends on BR2_ENABLE_LOCALE # linux-pam
> 	depends on BR2_USE_WCHAR # linux-pam
> +	depends on !BR2_STATIC_LIBS # linux-pam
> 	depends on !BR2_TOOLCHAIN_USES_MUSL # linux-pam
> 	depends on BR2_USE_MMU # fork(), linux-pam
> 	select BR2_PACKAGE_LINUX_PAM
> --
> 2.14.1

Reviewed-by: Carlos Santos <casantos@datacom.ind.br>
Arnout Vandecappelle Sept. 19, 2017, 8:06 p.m. UTC | #2
On 19-09-17 07:17, Baruch Siach wrote:
> Commit f52af619185 (util-linux: add menu items for login utilities)
> forgot to propagate the !BR2_STATIC_LIBS dependency of linux-pam to
> BR2_PACKAGE_UTIL_LINUX_LOGIN. Fix this.

 Note that the comment was already correct.

> 
> Fixes:
> http://autobuild.buildroot.net/results/e47/e472891a656e40f33081d9a5db1b1b6d71c1389f/
> http://autobuild.buildroot.net/results/9e4/9e443720ca247292ea612851962c611a9bcd3450/
> http://autobuild.buildroot.net/results/88f/88f4cecd7e2844701b0e75279c112e37283fc7f9/
> 
> Cc: Carlos Santos <casantos@datacom.ind.br>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>

 Applied to master, thanks.

 Regards,
 Arnout

> ---
>  package/util-linux/Config.in | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/package/util-linux/Config.in b/package/util-linux/Config.in
> index 80aab881a0da..5a600c333db3 100644
> --- a/package/util-linux/Config.in
> +++ b/package/util-linux/Config.in
> @@ -173,6 +173,7 @@ config BR2_PACKAGE_UTIL_LINUX_LOGIN
>  	bool "login"
>  	depends on BR2_ENABLE_LOCALE # linux-pam
>  	depends on BR2_USE_WCHAR # linux-pam
> +	depends on !BR2_STATIC_LIBS # linux-pam
>  	depends on !BR2_TOOLCHAIN_USES_MUSL # linux-pam
>  	depends on BR2_USE_MMU # fork(), linux-pam
>  	select BR2_PACKAGE_LINUX_PAM
>
diff mbox series

Patch

diff --git a/package/util-linux/Config.in b/package/util-linux/Config.in
index 80aab881a0da..5a600c333db3 100644
--- a/package/util-linux/Config.in
+++ b/package/util-linux/Config.in
@@ -173,6 +173,7 @@  config BR2_PACKAGE_UTIL_LINUX_LOGIN
 	bool "login"
 	depends on BR2_ENABLE_LOCALE # linux-pam
 	depends on BR2_USE_WCHAR # linux-pam
+	depends on !BR2_STATIC_LIBS # linux-pam
 	depends on !BR2_TOOLCHAIN_USES_MUSL # linux-pam
 	depends on BR2_USE_MMU # fork(), linux-pam
 	select BR2_PACKAGE_LINUX_PAM