Sriram Natarajan wrote: > > - On further poking around, I see no reason to bundle 'peardev'. I have > removed it.
Ok. > - There will be no man page for 'pear' / 'pecl' command. Hence, it is > not mentioned in the draft. There should be some documentation. But looking at 'man php' I'm reminded that it gives a listing of all the CLIs provided by php. So, since this case adds 2 more CLIs, that's a good place for saying something about them. There's not much to say in the case itself, but as an implementation detail, this integration should modify http://src.opensolaris.org/source/raw/sfw/usr/src/cmd/php5/Solaris/php.1.sunman to mention pear & pecl commands as well. > > Since it is tied to Apache 2.2 (delivering into /etc/apache/2.2/) > > shouldn't this be named SUNWapch22r-php5, to parallel the Apache > > package naming (SUNWapch22[dru])? > > > > > Thanks for catching it. > I believe that these package names should be consistent with the Apache > modules ARC draft that is being considered. What do you think ? Can we > go with the current package name (SUNWapch22m-php5) or should we need to > change it ? I don't have huge preferences on naming, but SUNWapch22m-php5 seems ok to me. -- Jyri J. Virkki - jyri.virkki at sun.com - Sun Microsystems
