On 06/29/09 10:44, Michal Pryc wrote:
Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-9555-v1/, fixes
9555 UM not working when there is network problem/some publishers not
available
The fix is to define a function in src/gui/misc.py to call refresh
and catch exceptions there.
Padraig,
Shouldn't we have some error dialog instead doing print?
This is a good question. We could display an error dialog for Package
Manager and Update Manager. It is not clear that we want to display an
error dialog if it occurs in UpdateManagerNotifier. Have you any
suggestions about what message we should give the user?
Based on johansen's comment in the bug this change should wait his
transport changes and the exception caught should be TransportError.
Padraig
Other then that this webrev looks good to me.
Michal
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
