Edit report at https://bugs.php.net/bug.php?id=39388&edit=1

 ID:                 39388
 Comment by:         tcallawa at redhat dot com
 Reported by:        judas dot iscariote at gmail dot com
 Summary:            provide --with-libzip-dir configure option to allow
                     usage of system's libzip
 Status:             Bogus
 Type:               Bug
 Package:            Zip Related
 Operating System:   Linux
 PHP Version:        5CVS-2006-11-05 (CVS)
 Assigned To:        pajoye
 Block user comment: N
 Private report:     N

 New Comment:

It doesn't look like 0.10 will work with PHP's zip extension out of the 
tarball, because it is still missing some necessary features that were never 
upstreamed (as far as I can tell). I've worked up a patch against 0.10 that 
adds the necessary functionality, and sent a copy to the libzip upstream for 
consideration. A copy of that patch can be found here:

http://spot.fedorapeople.org/libzip-0.10-php-changes.patch

Then, I reworked ext/zip/config.m4 to enable an option to use libzip (the 
system copy). That patch is here:

http://spot.fedorapeople.org/php-5.3.6-libzip.patch

I couldn't figure out a good way to test for overwrite support (the bits that 
the first patch add), so the configure tests will pass on a system libzip 
without the php changes applied, but I don't think the compile will succeed (it 
definitely won't work properly).


Previous Comments:
------------------------------------------------------------------------
[2011-04-06 08:33:48] oeriksson at mandriva dot com

With the recent security flaws related to libzip it's desirable to enable this 
option. Isn't the latest http://www.nih.at/libzip/libzip-0.10.tar.bz2 going to 
work fine now?

------------------------------------------------------------------------
[2006-11-05 19:21:28] paj...@php.net

There is no need of it. And no, you should *really* not use an external library.

The version bundled have more fixes and features than the released lib. Most of 
them are already in the libzip cvs, other not.

------------------------------------------------------------------------
[2006-11-05 03:50:59] judas dot iscariote at gmail dot com

Description:
------------
Currently there is no  --with-libzip-dir to easily allow distributions to use 
independantly packaged system libraries.

Reproduce code:
---------------
./configure --help | grep zip

Expected result:
----------------
--enable-zip        Include Zip read/write support.
--with-zlib-dir=DIR   zip: Set the path to libz install prefix.
--with-libzip=DIR     zip : path to libzip sources if not using the bundled 
library.

Actual result:
--------------
--enable-zip        Include Zip read/write support.
--with-zlib-dir=DIR   zip: Set the path to libz install prefix.


------------------------------------------------------------------------



-- 
Edit this bug report at https://bugs.php.net/bug.php?id=39388&edit=1

Reply via email to