On 02/05/09 12:38, Michal Pryc wrote: > 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?)
No. I will remove it when committing it. Padraig > > 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
