Niall, That's correct. After adding the new group entry to the file, compilation failed with a pkg format error. The error suggested to run pkgformat with no options to fix the file in-place, so I did. Looks like it just re-organized some lines and adjusted the ordering of some attributes of some the lines.
-ethan (sent from my phone.) On Apr 23, 2012, at 2:31 AM, Niall Power <[email protected]> wrote: > Hi Ethan, > > Looks good to me. > > I assume that the non-relevant changes to > system-install-auto-install-auto-install-common.mf were due to pkgfmt trying > to be helpful, right? > > Thanks, > Niall > > > On 04/23/12 04:25 AM, Ethan Quach wrote: >> Can I get a review of the following fixes. >> >> >> BugIDs: >> ---------- >> 7162980 <http://monaco.us.oracle.com/detail.jsf?cr=7162980> allow aiuser to >> write to AI client's config profile directory >> 7111645 <http://monaco.us.oracle.com/detail.jsf?cr=7111645> useradd next uid >> after install different in global zone and non global zone >> 7162573 <http://monaco.us.oracle.com/detail.jsf?cr=7162573> aiuser user >> definition should not be a role. >> >> >> Webrev: >> ---------- >> https://cr.opensolaris.org/action/browse/caiman/equach/aiuser/webrev/ >> >> >> Testing: >> --------- >> On a freshly installed system, verified that 'aiuser' is no longer a role. >> On a freshly installed system, verified that 'aiuser' now has uid 61 and gid >> 61. >> On a system updated to new package, verified that 'aiuser' has updated uid >> and gid with no issues. >> Built an AI image and created an install service from it. Installed a >> client using this image and verified that a derived manifest script can >> write to the config profile directory. >> >> >> >> thanks, >> -ethan >> >> >> >> _______________________________________________ >> caiman-discuss mailing list >> [email protected] >> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss > _______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
