Danek Duvall wrote:
    http://cr.opensolaris.org/~dduvall/pkg-misc2/

contains fixes for

    412 depot will not start
    1230 UnboundLocalError traceback from user action: local variable 'pw' 
referenced before assignment
    1272 IPS packages are missing files with "$" in their names

user.py
    Isn't the last pw.unlockfile() in this code trying to reference 'pw'
    from a different scope - I don't see 'pw' declared outside of the
    try/except block?

                 except:
                         if "pw" in locals():
                                 pw.unlockfile()
                         raise
                 pw.unlockfile()
                 ^^

Can't say that I understand the code in SolarisPackageDirBundle.py, so I ignored that.

The rest looks okay to me.

Trev

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

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

Reply via email to