Sriram Natarajan wrote:
>
> > The text in this section doesn't make a very clear distinction between
> > the two CLIs being delivered, for which purpose is each one used. I
> > suggest adding a few sentences to that effect.
>   
> please take a look at the updated draft and let me know if the 
> explanation provided within addresses your concern.

The 'pecl' CLI detail looks good, but I didn't see explanation about
'pear' yet, add something about it. I also see you added a new
'peardev' CLI, so also add a little info about its usage as well.

All these CLIs will deliver man pages right? Also add a sentence
saying so to the same section.

> > So downloaded packages will be installed into this location?
>   
> you could use /etc/php5/pear.conf to  specify where you want to download 
> these extensions. 

I know, I meant default location. It wouldn't hurt to mention the
location can be changed in the config, if you want.

> 2.6 Integration with Apache 
>
>        To provide closer integration of with Apache, provide php 
>        specific configuration file(php.conf) under '/etc/apache/2.2/conf.d' 
>        directory. This file will be delivered within SUNWapch2r-php5 package.

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])?

>        Also, PHP5 module for Apache 2 web server, will be now delivered
>        within SUNWapch2m-php5 package and will depend on SUNWapch2u,

And here SUNWapch22m-php5, dependency SUNWapch22u?



> the bunch of files currently mentioned in /var/php5 are the dependent 
> files necessary for PEAR to operate. However, under this location  
> 'pear/pecl' creates dynamic data like .channels . So, we cannot have 
> them under /usr.

It's really unfortunate that all these projects are delivering so much
static content into /var .. that's just not right. But if it is how
these components have been designed, not too much we can do about it
in the short term.

Nit: There's still some mis-aligned columns in the exported interface
table, example below but there are more. Seems like a small thing but
it really makes for an easier ARC review when the presentation is good...

pecl extension                  Uncommitted
Memcache extension                              Uncommitted

-- 
Jyri J. Virkki - jyri.virkki at sun.com - Sun Microsystems

Reply via email to