Skip to contentSkip to conversation

dillo: zlib is a mandatory dependency

PATCH New Gustavo Zacarias 2 participants 2 messages Jan 8, 2015

lore
Share & tools
lore

Gustavo Zacarias

author
#0 lore raw

PATCH 1/1 dillo: zlib is a mandatory dependency

Fixes:
http://autobuild.buildroot.net/results/10f/10f7ba386a13753c942139b8bc13ca74d8932f3a/

Also reorder variables a bit and ditch duplicate addition.

Signed-off-by Gustavo Zacarias <[email protected]>
---
 package/dillo/Config.in | 1 +
 package/dillo/dillo.mk  | 7 ++-----
 2 files changed, 3 insertions(+), 5 deletions(-)
package/dillo/Config.in +1 -0
index 5c2cc62..d06f2d1 100644
@@ -5,6 +5,7 @@ config BR2_PACKAGE_DILLO
 	depends on BR2_USE_MMU # fltk fork()
 	select BR2_PACKAGE_FLTK
 	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
+	select BR2_PACKAGE_ZLIB
 	help
 	  Dillo is a multi-platform graphical web browser known
 	  for its speed and small footprint.
package/dillo/dillo.mk +2 -5
index c216322..9bdd56f 100644
@@ -11,13 +11,10 @@ DILLO_LICENSE = GPLv3+
 DILLO_LICENSE_FILES = COPYING
 # For 0001-configure.ac-change-fltk-config-test-to-be-more-cros.patch
 DILLO_AUTORECONF = YES
-
-DILLO_DEPENDENCIES = fltk
-
+DILLO_DEPENDENCIES = fltk zlib \
+	$(if $(BR2_PACKAGE_LIBICONV),libiconv)
 DILLO_CONF_ENV = ac_cv_path_FLTK_CONFIG=$(STAGING_DIR)/usr/bin/fltk-config
 
-DILLO_DEPENDENCIES += $(if $(BR2_PACKAGE_LIBICONV),libiconv)
-
 ifeq ($(BR2_PACKAGE_OPENSSL),y)
 	DILLO_CONF_OPTS += --enable-ssl
 	DILLO_DEPENDENCIES += openssl
Signature · 1 line
2.0.5
Applied
#1 lore raw

PATCH 1/1 dillo: zlib is a mandatory dependency

Dear Gustavo Zacarias,

Quoted 10 lines · Gustavo Zacarias

Fixes:
http://autobuild.buildroot.net/results/10f/10f7ba386a13753c942139b8bc13ca74d8932f3a/

Also reorder variables a bit and ditch duplicate addition.

Signed-off-by Gustavo Zacarias <[email protected]>
---
 package/dillo/Config.in | 1 +
 package/dillo/dillo.mk  | 7 ++-----
 2 files changed, 3 insertions(+), 5 deletions(-)

Applied, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com