Looks good to me.
JR
On 16/03/2011 15:32, Padraig O Briain wrote:
I have created a new webrev,
http://cr.opensolaris.org/~padraig/ips-17326-v2/.
Padraig
On 03/16/11 11:31 AM, John Rice wrote:
Padraig - few comments:
- I would scroll to the added publisher and highlight it once the add
dialog goes.
- I would put an if statement around the setup of the new pub
checking on self.new_pub != None. (#476)
JR
On 11/03/2011 15:40, Padraig O Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-17326-v1/, fixes
17326 PM--can not reload repository list when add a new repository
The problem was that when a publisher was added from the Manage
Publishers dialog the list of publishers was not updated to include
the added publisher.
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
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
