diff mbox series

package/lynx: depends on dynamic lib

Message ID 20180203170555.1431-1-mlang@blind.guru
State Superseded
Headers show
Series package/lynx: depends on dynamic lib | expand

Commit Message

Mario Lang Feb. 3, 2018, 5:05 p.m. UTC
This fixes the following autobuild errors:
  http://autobuild.buildroot.net/results/9a7b37264d592add75dd2a0f233a2de4eba4f489
  http://autobuild.buildroot.net/results/d3ecc533b5c5a33a9385e7f1f689ddf9bfa6fd43

Signed-off-by: Mario Lang <mlang@blind.guru>
---
 package/lynx/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Yann E. MORIN Feb. 3, 2018, 5:17 p.m. UTC | #1
Mario, All,

On 2018-02-03 18:05 +0100, Mario Lang spake thusly:
> diff --git a/package/lynx/Config.in b/package/lynx/Config.in
> @@ -1,5 +1,6 @@
>  config BR2_PACKAGE_LYNX
>  	bool "lynx"
> +	depends on !BR2_STATIC_LIBS
>  	depends on BR2_USE_MMU

Software dependencies (!static) should go after hardware dependencies
(here, MMU).

Regards,
Yann E. MORIN.
Thomas Petazzoni Feb. 3, 2018, 5:46 p.m. UTC | #2
Hello,

On Sat,  3 Feb 2018 18:05:55 +0100, Mario Lang wrote:
> This fixes the following autobuild errors:
>   http://autobuild.buildroot.net/results/9a7b37264d592add75dd2a0f233a2de4eba4f489
>   http://autobuild.buildroot.net/results/d3ecc533b5c5a33a9385e7f1f689ddf9bfa6fd43
> 
> Signed-off-by: Mario Lang <mlang@blind.guru>
> ---
>  package/lynx/Config.in | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/package/lynx/Config.in b/package/lynx/Config.in
> index a338f83616..570d3fd15d 100644
> --- a/package/lynx/Config.in
> +++ b/package/lynx/Config.in
> @@ -1,5 +1,6 @@
>  config BR2_PACKAGE_LYNX
>  	bool "lynx"
> +	depends on !BR2_STATIC_LIBS
>  	depends on BR2_USE_MMU
>  	select BR2_PACKAGE_SLANG if !BR2_PACKAGE_NCURSES
>  	select BR2_PACKAGE_OPENSSL if !BR2_PACKAGE_GNUTLS
> @@ -7,3 +8,6 @@ config BR2_PACKAGE_LYNX
>  	  The Text Web-Browser.
>  
>  	  http://lynx.invisible-island.net/
> +
> +comment "lynx needs a toolchain w/ dynamic lib"
> +	depends on BR2_STATIC_LIBS

The comment needs a "depends on BR2_USE_MMU" dependency.

Thanks!

Thomas
diff mbox series

Patch

diff --git a/package/lynx/Config.in b/package/lynx/Config.in
index a338f83616..570d3fd15d 100644
--- a/package/lynx/Config.in
+++ b/package/lynx/Config.in
@@ -1,5 +1,6 @@ 
 config BR2_PACKAGE_LYNX
 	bool "lynx"
+	depends on !BR2_STATIC_LIBS
 	depends on BR2_USE_MMU
 	select BR2_PACKAGE_SLANG if !BR2_PACKAGE_NCURSES
 	select BR2_PACKAGE_OPENSSL if !BR2_PACKAGE_GNUTLS
@@ -7,3 +8,6 @@  config BR2_PACKAGE_LYNX
 	  The Text Web-Browser.
 
 	  http://lynx.invisible-island.net/
+
+comment "lynx needs a toolchain w/ dynamic lib"
+	depends on BR2_STATIC_LIBS