Brock Pytlik wrote:

> https://cr.opensolaris.org/action/browse/pkg/bpytlik/7136706-v2

image.py:

  - line 26: why not in alphabetical order with the rest of the imports?

  - line 3505: I don't understand why you're getting rid of the
    KeyboardInterrupt handling?  You don't have to return actdict if the
    unlinks fail and we got here for other reasons, but I think if we hit a
    ^C at this point, we actually want it to take effect.

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

Reply via email to