Ethan Quach wrote:
> Evan,
> 
> be_rename.c - 125 - nit - can you change this debug output to
> start with "be_rename: " like the others.

OOPS! fixed

> 
> libbe.h - 107 - add this to the end of the enumeration.  Recall
> we have a copy of this in beadm/messages.py

moved

-evan

> 
> 
> thanks,
> -ethan
> 
> 
> Evan Layton wrote:
>> Hi,
>>
>> I need to get a review the following fix. This simply adds a check
>> to see if we're attempting to rename the current BE and if so we
>> return a more appropriate error.
>>
>> 3332 beadm rename error message remains unhelpful
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=3332
>>
>> Webrev:
>> http://cr.opensolaris.org/~evanl/3332/
>>
>> Thanks!
>> -evan
>> _______________________________________________
>> caiman-discuss mailing list
>> caiman-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss


Reply via email to