Ethan,

Nit:
 289                     "zonepath dataset (%s) has no zone root container 
dataset, "

"has no" isn't proper, suggest "does not have".

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

Reply via email to