Tim Knitter wrote:
> 
> Ethan,
> 
> Nit:
>  289                     "zonepath dataset (%s) has no zone root container 
> dataset, "
> 
> "has no" isn't proper, suggest "does not have".

Fixed, thanks,

-ethan

> 
> I also agree with Evan about the "underneath it" comment.
> 
> Tim
> 
>> Need a review ...
>>
>> Defect:
>> ----------
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=4794
>>
>> Webrev:
>> ------------
>> http://cr.opensolaris.org/~equach/webrev.4794
>>
>>
>> I've reproduced the bug case and tested that these
>> changes fix the issue.  A user can now create a new
>> boot environment while an unsupported ipkg zone
>> exists on the system.  The unsupported zone is properly
>> skipped.
>>
>> The changes also fixes other functional cases that
>> also fail when there are unsupported zones in the
>> boot environment, such as mounting/unmounting
>> and destroying a boot environment with an unsupported
>> zone.
>>
>>
>> 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

Reply via email to