2012/5/26 Stuart Henderson <s...@spacehopper.org> > If this is added then there's no point in keeping pecl-zip. > > But then, is there any point in adding this when we have pecl-zip already? >
Actually pecl-zip and php-zip have the same upstream source coder and almost share the same code. But pecl-zip in portstree is in version 1.10.2 which was released about 3 years ago, and there are some bug fixed after that: https://github.com/php/php-src/tree/master/ext/zip So for me I would prefer php-zip than pecl-zip. wen > > > On 2012/05/26 16:30, wen heping wrote: > > Hi, > > > > Here is an patch to add -zip extension to lang/php port. php5-zip is > > required by the update of www/moodle which I am working. > > > > It build well on i386. > > > > OK? Comments? > > > > wen > > > ? php-zip-ext.diff.txt > > Index: Makefile.inc > > =================================================================== > > RCS file: /cvs/ports/lang/php/Makefile.inc,v > > retrieving revision 1.28 > > diff -u -r1.28 Makefile.inc > > --- Makefile.inc 28 Apr 2012 08:34:56 -0000 1.28 > > +++ Makefile.inc 26 May 2012 08:26:27 -0000 > > @@ -300,6 +300,12 @@ > > LIB_DEPENDS-xsl= textproc/libxslt > > WANTLIB-xsl= ${MODLIBICONV_WANTLIB} m xml2 z xslt>=3 exslt > > > > +# zip > > +MULTI_PACKAGES+= -zip > > +COMMENT-zip= zip functions for php5 > > +CONFIGURE_ARGS+= --enable-zip=shared,${LOCALBASE} > > +WANTLIB-zip= m z > > + > > # some variables to substitute > > SUBST_VARS= MODPHP_CONFIG_PATH PV ^MODULE_NAME > > PHP_VERSION= ${V} > > Index: 5.2/pkg/PLIST-zip > > =================================================================== > > RCS file: 5.2/pkg/PLIST-zip > > diff -N 5.2/pkg/PLIST-zip > > --- /dev/null 1 Jan 1970 00:00:00 -0000 > > +++ 5.2/pkg/PLIST-zip 26 May 2012 08:26:29 -0000 > > @@ -0,0 +1,8 @@ > > +@comment$ > > +@option no-default-conflict > > +@conflict php-zip->=5.2,<5.3 > > +@conflict php5-zip->=5.2 > > +@pkgpath www/php5/extensions,-zip > > +lib/php-${PV}/modules/zip.so > > +share/examples/php-${PV}/zip.ini > > +@sample ${SYSCONFDIR}/php-${PV}.sample/zip.ini > > Index: 5.3/pkg/PLIST-zip > > =================================================================== > > RCS file: 5.3/pkg/PLIST-zip > > diff -N 5.3/pkg/PLIST-zip > > --- /dev/null 1 Jan 1970 00:00:00 -0000 > > +++ 5.3/pkg/PLIST-zip 26 May 2012 08:26:29 -0000 > > @@ -0,0 +1,7 @@ > > +@comment $OpenBSD$ > > +@option no-default-conflict > > +@conflict php-zip->=5.3,<5.4 > > +@conflict php5-zip->=5.2 > > +lib/php-${PV}/modules/zip.so > > +share/examples/php-${PV}/zip.ini > > +@sample ${SYSCONFDIR}/php-${PV}.sample/zip.ini > > Index: files/DESCR-zip > > =================================================================== > > RCS file: files/DESCR-zip > > diff -N files/DESCR-zip > > --- /dev/null 1 Jan 1970 00:00:00 -0000 > > +++ files/DESCR-zip 26 May 2012 08:26:29 -0000 > > @@ -0,0 +1,7 @@ > > +Zip is a file format used for data compression and archiving. > > +A zip file contains one or more files that have been compressed, > > +to reduce file size, or stored as is. The zip file format permits > > +a number of compression algorithms. > > + > > +This PHP extension provides a processor independent API to Zip > > +transformations. > >