Thanks Glenn. Glad you explained everything. Only one more comment.
>
> Really? It verifies fine for me in netbeans (I specifically tried to
> make this PEP8 compliant). How is it not?
>
The indentation looks like it's still 8 spaces. I believe it should be 4.
It also looks like there are some lines > 79 characters.
Not sure if the following is a PEP8 issue or pylint. From what I've read
though,
supposedly to be truly PEP8 you need to be pylint clean also. Not clear we
want that extreme conformance at this point in time.
execfile("/usr/lib/python2.4/vendor-packages/osol_install/distro_const/" \
"DC_defs.py")
execfile('/usr/lib/python2.4/vendor-packages/osol_install/transfer_defs.py')
So the above kick out all kinds of errors. I found that doing something
like this
helps tremendously and is IMHO more correct (yeah I did the original
code so I'm
criticizing myself)
from osol_install.distro_const.DC_defs import IMAGE_INFO_FILE
Jean
>
>> vmc shell scripts:
>> I'll leave these for others
>>
>
> Fair enough. Thanks Jean!
>
>
