On 08/09/11 10:39, Danek Duvall wrote:
Tim Foster wrote:

https://cr.opensolaris.org/action/browse/pkg/timf/pkgdep_smf_symlinks/pkgdep_smf_symlinks-webrev

Looks good, just two small nits:

I'd probably make clear_cache() _clear_cache() instead, if it's essentially
a private method.

t_dependencies.py:

   - line 2418: "that behaviour" ->  "that the behaviour"

Don't you mean s/behaviour/behavior/ :-)

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

Reply via email to