Prashant Srinivasan wrote: > > Can someone review this patch from the upstream community? It updates
Not sure what you mean by "patch from the upstream community"? We're not reviewing that (nor do we need to), just your build changes. > http://cr.opensolaris.org/~psriniva/webrev-6721879-v2/ Seems ok to me, except remember to delete the old ruby-1.8.6.tar.bz2 now. I assume you've verified that nothing has changed interface-wise here, but it'd be good to point to the release notes in this review thread. -- Jyri J. Virkki - jyri.virkki at sun.com - Sun Microsystems
