Sue,
Applied all suggestions and re-issued webrev.
Thanks,
William
On 08/29/11 05:16 PM, Sue Sohn wrote:
On 08/29/11 11:11, William Schumann wrote:
The remaining fixes are in create_profile.py and delete_profile.py.
Multiple profiles are accepted;
exit status 1 if any serious errors are encountered, 0 otherwise.
https://cr.opensolaris.org/action/browse/caiman/wmsch/7034620/webrev/
Tested success, failure, combined success and failure on single
command line.
Unit tests unaffected.
Thanks,
William
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
Hi William,
create_profile.py
210 Can you expand this comment? Not clear what "assuming none" means
230 Shouldn't there be a continue after this line? If the file doesn't
exist, we shouldn't try to open it.
339 should add_errors be set here along with printing an error message?
Sue
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
