Prashant Srinivasan wrote:
>
> Hello,
> 
> http://cr.opensolaris.org/~chriszhu/ruby-webrev/ is the webrev for the 
> inclusion of Ruby 1.8 into Solaris(PSARC/2007/600).

- usr/src/Targetdirs doesn't list many dirs added by the packages

- "patchs" - I'd be a little happier if you spell out "patches"...

- Do the pkg 'copyright' files need any Sun preambles?

- SUNWruby18u doesn't work in IPS due to custom scripting ;-(



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

Reply via email to