Padraig - looks good, few minor comments:

1634: change > to != to make intent clearer
1678: add brackets to clarify precedence to reader
2837: use global here instead of 0

JR


On 20/05/2011 11:02, Padraig O Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-18000-v4/, fixes
18000 packagemanager should be enhanced to know about system publishers.

The Manage Publishers dialog is updated to handle a system publisher. The fact that it is a system publisher is noted in the details panel, the Modifyy, Remove, Up and Down buttons are disabled and changing sticky or enabled status us not permitted.

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

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

Reply via email to