Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] php: fix build breakage
@ 2011-11-25 15:49 Gustavo Zacarias
  2011-11-26 12:56 ` Peter Korsgaard
  0 siblings, 1 reply; 2+ messages in thread
From: Gustavo Zacarias @ 2011-11-25 15:49 UTC (permalink / raw)
  To: buildroot

The zip extension requires zlib, so select it.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/php/Config.ext |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/package/php/Config.ext b/package/php/Config.ext
index 4f1614e..6df8cc3 100644
--- a/package/php/Config.ext
+++ b/package/php/Config.ext
@@ -125,6 +125,8 @@ config BR2_PACKAGE_PHP_EXT_SYSVSHM
 
 config BR2_PACKAGE_PHP_EXT_ZIP
 	bool "zip"
+	select BR2_PACKAGE_PHP_EXT_ZLIB
+	select BR2_PACKAGE_ZLIB
 	help
 	  Zip read/write support
 
-- 
1.7.3.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH] php: fix build breakage
  2011-11-25 15:49 [Buildroot] [PATCH] php: fix build breakage Gustavo Zacarias
@ 2011-11-26 12:56 ` Peter Korsgaard
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Korsgaard @ 2011-11-26 12:56 UTC (permalink / raw)
  To: buildroot

>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 Gustavo> The zip extension requires zlib, so select it.

 Gustavo> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
 Gustavo> ---
 Gustavo>  package/php/Config.ext |    2 ++
 Gustavo>  1 files changed, 2 insertions(+), 0 deletions(-)

 Gustavo> diff --git a/package/php/Config.ext b/package/php/Config.ext
 Gustavo> index 4f1614e..6df8cc3 100644
 Gustavo> --- a/package/php/Config.ext
 Gustavo> +++ b/package/php/Config.ext
 Gustavo> @@ -125,6 +125,8 @@ config BR2_PACKAGE_PHP_EXT_SYSVSHM
 
 Gustavo>  config BR2_PACKAGE_PHP_EXT_ZIP
 Gustavo>  	bool "zip"
 Gustavo> +	select BR2_PACKAGE_PHP_EXT_ZLIB
 Gustavo> +	select BR2_PACKAGE_ZLIB

Thanks, but selecting the zlib extension doesn't seem completely right
(not a big deal, but still), so I've changed it to only select zlib +
add the --with-zlib / add zlib to deps in the .mk instead.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-11-26 12:56 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-25 15:49 [Buildroot] [PATCH] php: fix build breakage Gustavo Zacarias
2011-11-26 12:56 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox