John Rice wrote:
Shawn Walker wrote:
John Rice wrote:
Hi - this webrev adds logging handlers to the PM and UM. For now they
are just outputting to tmp files:
/var/tmp/packagemanager_info.log
/var/tmp/packagemanager_error.log
var/tmp/updatemanager_info.log
/var/tmp/updatemanager_error.log
Post 2010.03 we can look at hooking them into the GUI so the user can
view them directly from the GUI, but this will be tracked as a
separate enhancement from this bug.
http://cr.opensolaris.org/~jmr/pm_12240_pm_logging_10Dec_940pm/
12240 PM / UM need to intercept and display client messages
The only thing I'd like to see is a suggestion by the GUI to check
these log files for more information when operations fail.
Shawn - we need any API calls which emit Errors to give an appropriate
return code, if they are not going to throw an Exception. Then we know
What does "return code" mean here? As I noted before, you already know
if something is an "info" message, "warning" message, or "error" message
based on the message context which is provided to you.
So I'd like to land what we have, document the log files in the Help and
work with you folks to get suitable return codes in place over the
coming months so we can handle this in a robust fashion in the GUI. If
You'd have to clarify what you mean by "return codes". The messages
returned by the API are intended to be used as is, although the text of
some of them may need to be a bit more generic.
we have a failure in an API call and an error is emitted to the log one
possibility is to flag the condition in the Status Bar as we did for
Search failures and allow the user to click on an error icon to bring up
the logs for them to review. Similar functionality exits in NetBeans and
Firefox to signal none fatal errors to the user. It will be post 2010.03.
That would be good. Giving users access to these messages is really the
main thing. It is especially important when things go wrong and we need
to figure out why.
Cheers,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
