Jyri Virkki wrote:

These are basically nits. 
+1 otherwise.
> 1.2     Previous Relevant ARC cases
>
>         PSARC/2007/641 - Enabling built-in extensions. 
>         PSARC/2007/552 - Upgrading PHP 5.2.4
>         PSARC/2007/470 - Upgrading PHP 5.2.3
>         PSARC/2007/168 - Integrating PHP 5.2.0
>         PSARC/2008/538 - Integrating PHP 5.2.6
>   
I'm sure you mean LSARC/2008/538..  It seems odd that it switched from 
PSARC->LSARC.  <shrug>.  I'm still trying to figure out where that line 
is, and continue to be surprised.  I'm sure I'll get it eventually.
>
>         /usr/php/5.2/modules/ncurses.so
>         /usr/php/5.2/modules/odbc.so
>   

When did the directory structure change?  I followed this forward from 
the original PSARC/2007/168 case, and there the convention was 
"/usr/php5/$(VER)".  This seems to be the first follow-on case where the 
directory structure is indicated differently..

> 2.9     Package Dependencies
>         
>         SUNWphp52u-ncurses will depend on SUNWncurses
>         
>         SUNWphp52u-odbc will depend on SUNWunixodbc
>         
>         SUNWphp52u-snmp will depend on SUNWnetsnmp-core
>   
Why isn't this last dependency listed in the imported interface 
section?  LSARC/2008/538 is a cleaner read on the package->library 
dependency.

Reply via email to