Hey Brock,

On Sun, 2011-04-17 at 13:43 -0700, Brock Pytlik wrote:
> Webrev:
> http://cr.opensolaris.org/~bpytlik/ips-18172-v1/

Looks good - one suggestion, for src/modules/publish/dependencies.py
line 542, could you add to the docstring explaining why we're returning
the new (name, pfmri) tuple?

        cheers,
                        tim

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

Reply via email to