Hi Siva, Jyri, I have made the changes reqd (including the appropriate #ident). I ran wx pbchk which returned no errors.
Kindly review the code changes. http://cr.opensolaris.org/~gs193864/cr6644643/ Thanks, Gopal. Sivakumar Shanmugasundaram wrote: > Gopal, > > Looks good to me. I assume you have done 'wx pbchk' on your wokspace and > do not get any errors from this command. > Oh, I do not see Jyri's comments addressed. Please fix them. Add the > appropriate '#ident' and do 'wx redelget' > > Siva > > > Gopalakrishnan Sankaran wrote: > >> Hi Siva, >> >> Have made the changes. Please review it again. >> http://cr.opensolaris.org/~gs193864/cr6644643/ >> >> Thanks, >> Gopal. >> >> Sivakumar Shanmugasundaram wrote: >> >>> Gopal, >>> >>> A couple of comments. >>> >>> Please change the Webrev comments to have the 'exact' bug synopsis >>> against all the files. >>> >>> You could do a 'wx pbchk' to find errors >>> >>> postinstall-ruby: >>> >>> You could use another file name instead of just 'temp'. May be use >>> rbconfig.$$ or similar. >>> >>> Other changes look good to me (wx pbchk - please). >>> >>> Siva >>> >>> >>> Gopalakrishnan Sankaran wrote: >>> >>> >>>> Hi, >>>> >>>> Here is the webrev after making the changes required (as suggested in >>>> the initial round of review) >>>> Kindly review the changes. >>>> >>>> http://cr.opensolaris.org/~gs193864/cr6644643/ >>>> >>>> >>>> >>> _______________________________________________ >>> >>> >>> webstack-discuss mailing list >>> webstack-discuss at opensolaris.org >>> http://mail.opensolaris.org/mailman/listinfo/webstack-discuss >>> >>> >> > _______________________________________________ > > > webstack-discuss mailing list > webstack-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/webstack-discuss > -- Thanks, Gopal http://blogs.sun.com/gkrish