Bug #39388 [Com]: provide --with-libzip-dir configure option to allow usage of system's libzip

Edit report at https://bugs.php.net/bug.php?id=39388&edit=1 ID: 39388 Comment by: tk at giga dot or dot at Reported by: crrodriguez at opensuse dot org Summary: provide --with-libzip-dir configure option to allow usage of system's libzip Status: Not a bug 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: The current libzip HEAD revision from the mercurial repository contains AFAICT all necessary changes for PHP. Please let me know if it doesn't. Previous Comments: ------------------------------------------------------------------------ [2011-09-28 05:35:25] glen at delfi dot ee i updated the patch to really link with system libzip and not to link with libz (as that it is libzip dependency not zip ext in case of system libzip) http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/packages/php/system-libzip.patch? sortby=date i would reformat the patch indent if there's any interest it getting applied upstream, it's currently "broken" indent just to get diffs better to understand (and update with new upstream changes) ------------------------------------------------------------------------ [2011-07-20 17:40:51] crrodriguez at opensuse dot org tcallawa at redhat dot com : Great, patches look OK, will try them asap in openSUSE packages. ------------------------------------------------------------------------ [2011-07-20 15:54:40] tcallawa at redhat dot com 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). ------------------------------------------------------------------------ [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] pajoye@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. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=39388 -- Edit this bug report at https://bugs.php.net/bug.php?id=39388&edit=1