Hi, Thanks for your information. I have just changed the NO_COMMENT string, etc. according to your feedback bellow. Please review the re-generated webrev for the fixes at: http://cr.opensolaris.org/~luojiac/6638783/
Thanks! Luojia(Jenny) Chen Software Engineer ISV Engineering v-mail:(510) 550-2394 SUN Microsystems ----- Original Message ----- From: Jyri Virkki <jyri.vir...@sun.com> Date: Tuesday, December 18, 2007 7:28 pm Subject: Re: [webstack-discuss] Fix for bug# 6638783 To: Luojia Chen <Luojia.Chen at Sun.COM> Cc: webstack-discuss at opensolaris.org > Luojia Chen wrote: > > > > Hi, > > > > I just post the webrev for 6638783 at: > > http://cr.opensolaris.org/~luojiac/bugfix/ > > > > Can you please review this? > > > Prepared by: Super-User (root) on Mon Dec 17 17:10:30 PST 2007 > > Looks like you're running as root for your development work. Not good > practice! > > The code changes look ok. > > Some details to fix: > > Run "wx ea" and change the NO_COMMENT strings to the revision change > comment, which in this case is: > > "6638783 following manpage instructions results in mysql user with > inappropriate $HOME" > > (but without the quotes) > > After that run "wx redelget". This will modify the checkin comments > for all files to the sfw practice of nothing but "bugid > bugdescription". > Then re-run "wx webrev" and update the URL above with the regenerated > webrev output. > > > -- > Jyri J. Virkki - jyri.virkki at sun.com - Sun Microsystems > _______________________________________________ > webstack-discuss mailing list > webstack-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/webstack-discuss