Re: [alpine-devel] [PATCH] main/php: improved APKBUILD, see changes [resent, updated]
On Sunday, January 9, 2011 8:53am, "Natanael Copa" <ncopa_at_alpinelinux.org> said:
> On Fri, 7 Jan 2011 13:33:47 -0600
> Matt Smith <mcs_at_darkregion.net> wrote:
>> I based my updates off of Arch Linux's work for this package.
> Thanks alot for working on this. I have felt i dont have capacity for
> doing it myself so I am very happy that you do it and that you deliver
> such high quality.
No problem, I'm glad to help.
>> - Added php-embed subpackage.
> Is this needed? If not, can we add it when its needed or on request in
Not really, however, it is simple to build and install. Plus, we may win
some Arch users with its presence. :)
>> - Removed php-cli subpackage; it's now in the main 'php' package.
> This means upgrades will break and that wiki docs needs updates.
> Please try install alpine 2.1.4 in a virtual with php+lighttpd and have
> a simple <?php phpinfo(); ?> script.
> Then upgrade to edge and see what happens.
> (you might need add some replaces="..." in there)
> What do you think about keeping the php-cli package as separate
> package, have the php as an empty meta package which has php-cgi as
> depends (for not breaking upgraders), and have a php-common package
> that holds the php.ini and shared files.
That's a good idea; I'll set it up shortly.
>> - Added the configure options --with-readline and --enable-pcntl to
>> the 'php' package (previously known as the php-cli package).
> Is readline really needed? How often is php used from command line? You
> dont need the readline support for running scripts.
> I'm not saying I'm against readline, I'm just thinking is it worth
> the extra "bloat"?
It might come in handy. In my updated APKBUILD, I'll make it so that
readline is only enabled for php-cli.
Received on Thu Jan 13 2011 - 00:03:28 UTC