Padraig O'Briain wrote:


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.

Maybe we can call some parent.handle_exception function. So the consumer (PM UM or Webinstall) will decide what to do with it. Or simply re-raise that exception and catch it on the consumer side?

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

Reply via email to