[...]
>>
>>>
>>> 1777: This error "Couldn't unmount target BE" still looks cryptic. 
>>> can we come up with a more meaningful error message?
>>
>> Sure. I have changed the message to:
>>
>> "Couldn't unmount target boot environment. It is left mounted on 
>> '/a'\n"));
>>
>> However, I am open to suggestions :-)
> There are other error messages from orchestrator and libbe if the 
> failure happens. Please remove "It is left mounted on '/a'" and retain 
> "Couldn't unmount target boot environment."

I have changed the message, also taking into account
Dave's suggestion (got rid of contraction).
If you would like to check, the webrev has been updated
accordingly:

http://cr.opensolaris.org/~dambi/ai-err-report-no-pep8-cr-diff
http://cr.opensolaris.org/~dambi/ai-err-report-no-pep8-cr-full

Thank you,
Jan


Reply via email to