Ethan Quach wrote: > I've updated the webrev with a TODO. To track this, I can file > another defect to track removal of this code. That defect will > hinge on 617643. >
That's 6176743. > > -ethan > > > Virginia Wray wrote: > >> 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 >> >> >> >> >> >> >> >> > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >
