Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:37670
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain chiaraquartet.net from 38.99.98.18 cause and error)
Message-ID: <482CCFCF.7080202@chiaraquartet.net>
Date: Thu, 15 May 2008 19:05:35 -0500
User-Agent: Thunderbird 2.0.0.14 (X11/20080502)
MIME-Version: 1.0
To: internals Mailing List <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: phar enabled by default as of now
From: greg@chiaraquartet.net (Gregory Beaver)

Hi,

I just committed to ext/phar's config.m4/w32 to make phar enabled by
default as a shared extension.  This is a better default setting than
building phar in statically, as phar has optional dependencies on the
zlib and bz2 extensions to enable compressed phar archives.  If phar is
built statically, then it would require both zlib and bz2 to be built
statically in order to use them, something that is obviously not going
to happen, as they have been around a while and nobody has even
suggested it (to my knowledge).  Having phar shared means one can easily
plug in zlib/bz2 support without recompiling PHP.

I'm open to further input/tweaks on this decision.  If you have a better
idea, go ahead and commit to phar's config.m4/w32, these files are very
easy to change if needed.

Greg