Shawn Walker wrote:

>   http://cr.opensolaris.org/~swalker/pkg-15864/

Isn't this an incompatible API change?  At the very least, this is an API
change.

client.py:

  - line 600, et al: why not just move the "import warnings" down to the
    one place we use it, and not hack up each instance of a perfectly
    reasonable variable name just to work around a lint suggestion.

  - line 1055: why not 

        for c in "\\ \t\n'`;&()|^<>?*":

  - line 1101: I don't think you need this anymore.

  - line 4521: use "is"

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

Reply via email to