Padraig, LGTM, with one minor cosmetic thing: http://cr.opensolaris.org/~padraig/ips-6014-v2/src/gui/modules/repository.py.wdiff.html
Line: 354 (is this empty line needed?) Michal Padraig O'Briain wrote: > The webrev http://cr.opensolaris.org/~padraig/ips-6014-v1/ fixes > 6014 Would like ability to disable authorities rather than deleting them > 5202 Catalog is refreshed too often for Manage Repositories dialog > operations > > A check box is added to the repository list to allow enabling and > disabling of a repository. > All the rows have been made sortable but the data is initially presented > unsorted. > For 5202 we avoid refreshing the catalog if the only changes the user > made was to change the preferred authority and change it back to the > original preferred authority before closing. > > 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
