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"

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

Reply via email to