James Gates wrote:
>
>         4.5.2. Imported Interfaces
> 
>         NAME                           STABILITY       NOTES
>         PHP5                           Uncommitted     PSARC/2007/168
>         Apache2                        Uncommitted     PSARC/2004/676

The apache2 import should really be PSARC/2007/586 as that is the one
available in Nevada/OpenSolaris, where this case is headed (2004/676
is the old apache 2.0 which doesn't exist anymore).


>         4.5.3. Exported Interfaces
> 
>         NAME                           STABILITY       NOTES
> 
>         SUNWphppgadminu                committed       Usr component
>         SUNWphppgadminr                committed       Root component
>         Files Layout within pkg        uncommitted
> 
>         One config file is delivered here:
>               /etc/phppgadmin/config.inc.php

Not clear to me why the entire file layout (under
/usr/postgres/phppgadmin) should be a public interface. Do you intend
to document & supports users to change/tune/reference the content
there?

Without looking at it in detail, seems like everything below
/usr/postgres/phppgadmin is project private and
/etc/phppgadmin/config.inc.php is the public interface, no?

The top path "/usr/postgres/phppgadmin/" does need to be an
interface as well since users need to hardcode that path into their
web server configuration. But making everything under it private seems
to make sense (then you can update the version without worrying if any
implementation files move around).


>         One has to setup the webserver to activate this.

This seems to be the key missing bit here.

Have you evaluated the possibility delivering an additional
apache2-specific connector package which enables it (probably by
delivering appropriate apache2 configuration into
/etc/apache2/2.2/conf.d/) if the package is installed?  That would 
allow documenting a one-line procedure setting up phpPgAdmin 
("pkg install SUNWphpphadmin SUNWapch22-phppgadmin").

Merely delivering some php files doesn't add much value. Anyone could
download the phpPgAdmin tarball and untar it into their docroot about
as easily as typing "pkg install SUNWphpphadmin". The opportunity for
real value-add for OpenSolaris here is if you can also deliver
something that "just works". 

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

Reply via email to