Hi Ethan - The changes look ok to me also. Evan echoed my thought, which was...how will the fix for 617643 be tracked and then implemented. g
On 10/27/08 11:07, Evan Layton wrote: > Hi Ethan, > > The changes look fine to me. > > One nit/idea on the comments though. Since we're doing this fix as a work > around > for 6176743 do we want to add a "TODO:" here to state that this needs to be > revisited when 6176743 gets fixed? > > Thanks! > -evan > > > > Ethan Quach wrote: > >> Can I get a review one liner (plus comments) >> >> Webrev: >> ------------ >> http://cr.opensolaris.org/~equach/webrev.4041 >> >> >> Defect: >> ---------- >> 4041 Creating a BE with zones fails when an alternate BE is currently >> mounted >> http://defect.opensolaris.org/bz/show_bug.cgi?id=4041 >> >> >> With this fix, new boot environments can be created while >> some other boot environment is mounted with zones. >> >> >> thanks, >> -ethan >> >> _______________________________________________ >> caiman-discuss mailing list >> caiman-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >> > > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss > -- Ginnie -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://mail.opensolaris.org/pipermail/caiman-discuss/attachments/20081027/1856960a/attachment.html>
