Please include the WrapIndexingException condition as part of this fix.

What happens if the attempt to create a temp file fails? Please catch that exception and print the original traceback to the screen as is done now.

How could we get to line 2950 and have tb be the equivalent of True?

Also, it might be worth investigating whether we can use (or extend) the logger module so that it actually takes care of all of these issues for us.

Other comments:
Please use the same format as exists in the rest of the file. This means 8 space indents, continuation lines indented 4 spaces, etc.

Brock

John Sonnenschein wrote:
Ping ?


On 2009-11-10, at 12:02 PM, John Sonnenschein wrote:

http://cr.opensolaris.org/~error404/1498/

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

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

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

Reply via email to