Looks okay to me. Does the pkginfo.tmpl have a version number on its DESC= line, if so does that need the p230 bit appended?
Paul > -----Original Message----- > From: sfwnv-discuss-bounces at opensolaris.org > [mailto:sfwnv-discuss-bounces at opensolaris.org] On Behalf Of > Prashant Srinivasan > Sent: 03 July 2008 22:35 > To: sfwnv-discuss at opensolaris.org > Subject: [sfwnv-discuss] Code review for Ruby patch update(CR 6721879) > > [setting Reply-to to webstack-discuss] > > Can someone review this patch from the upstream community? > It updates Ruby to 1.8.6-p230. Importantly it houses a fix > for a reported Security vulnerability. > > http://cr.opensolaris.org/~psriniva/webrev-6721879-v2/ > > Thanks, > -ps > > > > > -- > Prashant Srinivasan > F/OSS Enthusiast > Sun Microsystems, Inc. > http://blogs.sun.com/prashant > GnuPG key: > http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x82FBDE5A > > _______________________________________________ > sfwnv-discuss mailing list > sfwnv-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/sfwnv-discuss >