Tim Foster wrote:

> > >>> http://cr.opensolaris.org/~timf/13799-webrev/
> 
> I've moved that statement as well, and updated the webrev. Thanks again.

This looks okay to me.  A couple of nits:

  - Put a space after a colon in dictionaries

  - Do we really need a warning in install_sw() if the extra_env argument
    is used?  Seems to me like it's debugging info.

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

Reply via email to