On 02/ 4/11 10:11 AM, John Fischer wrote:
Sue,

Thanks!!

On 02/ 4/11 09:37 AM, Sue Sohn wrote:
I'd suggest making '/etc/netboot' a constant in installadm_common.py and then
using here instead of the string.

Added.

Also, I'm not sure why you changed line 330, seems like the original is fine.

I think this is because I forgot to do a merge prior to making the change. I
have once again made the file pep8 clean. The changes removed this line.

The new webrev is located at:

http://cr.opensolaris.org/~johnfisc/7013975-list-sparc-clients-2/
http://cr.opensolaris.org/~johnfisc/7013975-list-sparc-clients-diff-2/

Thanks!!

John

John,

Looks ok now.

Sue
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to