Date: Thu, 13 Jan 2011 00:03:28 -0600 (CST)
Subject: Re: [alpine-devel] [PATCH] main/php: improved APKBUILD, see changes [resent, updated]
From: "Matt Smith" <mcs@darkregion.net>
To: "Natanael Copa" <ncopa@alpinelinux.org>
Cc: alpine-devel@lists.alpinelinux.org
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain;charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Importance: Normal
In-Reply-To: <20110109145352.739b3b64@alpinelinux.org>
References: <1294428827-14536-1-git-send-email-mcs@darkregion.net> 
 <20110109145352.739b3b64@alpinelinux.org>
Message-ID: <1294898608.246126205@192.168.4.58>

On Sunday, January 9, 2011 8:53am, "Natanael Copa" <ncopa@alpinelinux.org> =
said:=0A> On Fri,  7 Jan 2011 13:33:47 -0600=0A> Matt Smith <mcs@darkregion=
.net> wrote:=0A> =0A>> I based my updates off of Arch Linux's work for this=
 package.=0A> =0A> Thanks alot for working on this. I have felt i dont have=
 capacity for=0A> doing it myself so I am very happy that you do it and tha=
t you deliver=0A> such high quality.=0A=0ANo problem, I'm glad to help.=0A=
=0A...snip...=0A>> - Added php-embed subpackage.=0A> =0A> Is this needed? I=
f not, can we add it when its needed or on request in=0A> future?=0A=0ANot =
really, however, it is simple to build and install.  Plus, we may win=0Asom=
e Arch users with its presence.  :)=0A =0A>> - Removed php-cli subpackage; =
it's now in the main 'php' package.=0A> =0A> This means upgrades will break=
 and that wiki docs needs updates.=0A> =0A> Please try install alpine 2.1.4=
 in a virtual with php+lighttpd and have=0A> a simple <?php phpinfo(); ?> s=
cript.=0A> =0A> Then upgrade to edge and see what happens.=0A> =0A> (you mi=
ght need add some replaces=3D"..." in there)=0A> =0A> What do you think abo=
ut keeping the php-cli package as separate=0A> package, have the php as an =
empty meta package which has php-cgi as=0A> depends (for not breaking upgra=
ders), and have a php-common package=0A> that holds the php.ini and shared =
files.=0A=0AThat's a good idea; I'll set it up shortly.=0A=0A>> - Added the=
 configure options --with-readline and --enable-pcntl to=0A>> the 'php' pac=
kage (previously known as the php-cli package).=0A> =0A> Is readline really=
 needed? How often is php used from command line? You=0A> dont need the rea=
dline support for running scripts.=0A> =0A> I'm not saying I'm against read=
line, I'm  just thinking is it worth=0A> the extra "bloat"?=0A=0AIt might c=
ome in handy.  In my updated APKBUILD, I'll make it so that=0Areadline is o=
nly enabled for php-cli.=0A=0AMatt



---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---