[izpack-devel] FW: Logging

Markus Schlegel markus.schlegel at pulinco.com
Fri Dec 1 08:35:02 CET 2006


Hi Elmar
 
Maybe you have to consider to change the "Exception" argument into "Throwable", at least for "addError(..)".
If not, it might be nasty to log a catched java.lang.Error...
 
Markus

________________________________

Von: izpack-devel-bounces at lists.berlios.de im Auftrag von Elmar Grom
Gesendet: Do 30.11.2006 18:20
An: izpack-devel at lists.berlios.de
Betreff: Re: [izpack-devel] FW: Logging



Hi Klaus,

here is a snippet of the API.

Log.getInstance ()

public void addMessage (int        message,
                        String []  detail)

public void addWarning (int        message,
                        String []  detail,
                        Exception  exception)

public void addError (int        message,
                      String []  detail,
                      Exception  exception)


The integer identifiers for the messages are defined in the Log class. The
logger will pull the actual message text from the langpack when building the
report. In this way messages are localized without extra burden on the
calling code.

In addition, the caller can provide detail information to refine the
message. To support this feature, some messages might have variable parts
defined, such as a file name, path, some numbers, settings etc. These will
be inserted by the logger from the detail array into the localized text. For
each message it will be exactly defined if there are variable parts, what
they are and in which order the detail must be provided.

Messages are just for generic information, for example which panel was
presented or what option was chosen. Warnings and Errors are added when
things go wrong, therefore these two methods include the ability to attach
an exception. The logger will include the Exception.toString() result with
each message that had an associated exception.

It is not possible to use the identifier of an error message to add a
warning and so on, since the message identifiers are organized into groups
and each add method verifies that a legal value was used for the call before
it actually adds the message.

I hope this answers your questions

        Elmar


-----Original Message-----
From: Bartz, Klaus [mailto:Klaus.Bartz at coi.de]
Sent: Thursday, November 30, 2006 12:14 AM
To: elmar at grom.net; izpack-devel at lists.berlios.de
Subject: RE: [izpack-devel] FW: Logging


Hi Elmar,
is it possible to add an "identifier" to the message slot like

void Logger.getInstance().setMessage(String loggingSlot, String msg,
Throwable who)

or
public final static int ERROR = 0;
public final static int WARNING = 1;
public final static int MESSAGE = 2;
void Logger.getInstance().set(int type, String loggingSlot, String msg,
Throwable who)

loggingSlot and who can be null.

The dumping of slot contents for MESSAGE can be triggert by command line
(e.g. string of comma separated identifier).
In opposite to log4j (if I have understand it right) there is no hierarchy
else
special informations can be selective dumped out.
With such a mimik it is possible to log more informations without making the
log blind bend and discussions about logged informations can be easier
blocked.
It can be simple enough to impl it self in opposite to use log4j (I do not
like
log4j :-)).

Cheers

Klaus

>-----Original Message-----
>From: izpack-devel-bounces at lists.berlios.de
>[mailto:izpack-devel-bounces at lists.berlios.de]On Behalf Of Elmar Grom
>Sent: Wednesday, November 29, 2006 11:37 PM
>To: Izpack
>Subject: [izpack-devel] FW: Logging
>
>
>Klaus,
>
>you make a good point here. Users might not be happy about a load of
>messages, even if they are all good.
>
>The logger works as follows:
>
>1) log messages are posted by calls to the logger
>2) there are three categories of messages: messages, warnings
>and errors
>3) the logger simply collects the messages but takes no further action
>4) the logger provides information if any warnings or errors have been
>posted
>5) the installer uses this information to display an
>appropriate generic
>message (probably in the finish panels) -> (success, possible problems,
>install failed)
>6) a button will be provided to allow the user to save a
>detailed report
>7) when asked to save a report, the logger produces a report
>with generic
>information about the install plus all messages posted. The
>messages are
>sorted by category (message/warning/error) and they are localized.
>
>In this way the user is never bombarded with a flood of
>messages that might
>only succeed in leaving doubt about the success of the operation, even
>though they might be completely meaningless. If necessary the
>report text
>file may be e-mailed to the developer.
>
>Because of your input, I think it might be a good idea to
>allow developers
>to decide if logging is actually turned on and perhaps even to
>what level.
>This could be done by a top level switch in the xml file. Even
>if this is
>done, there might be cases where an installation problem might
>need to be
>debugged. For such cases logging might be forced on by a command line
>switch.
>
>Here is what a report looks like right now. The message and
>warning section
>would not show up at all if there is nothing recorded, I just
>wanted to show
>what it would look like.
>
>---------------------------------------------------------------
>-------------
>---
>                           IzPack INSTALLATION REPORT
>---------------------------------------------------------------
>-------------
>---
>
>WARNING!!! The installation did not succeed!
>
>This report contains 0 general message(s), 0 warning(s) and 1 error(s)
>
>Installation report for : ProScholar Version 2.2
>Install time            : 11-29-2006 [14:29:37]  Pacific Standard Time
>Install directory       : D:\ProgramFiles\ProScholar
>
>                         --- Installation Messages ---
>
>Message  [0] - ...
>
>                               --- Warnings ---
>
>Warning [0] - ...
>
>                                --- Errors ---
>
>Error [0] - unable to write 'D:\ProgramFiles\ProScholar\app.jar'
>---------------------------------------------------------------
>-------------
>---
>
>
>Cheers
>
>       Elmar
>
>-----Original Message-----
>From: Bartz, Klaus [mailto:Klaus.Bartz at coi.de]
>Sent: Wednesday, November 29, 2006 1:13 AM
>To: elmar at grom.net; izpack-devel at lists.berlios.de
>Subject: RE: [izpack-devel] Logging
>
>
>Hi Elmar,
>I think all developer of IzPack knows that there are some
>points in the product which are not optimal solved. Of course,
>we should solve it if possible and time is right on work-time-device.
>
>Some weeks ago I have had a little problem with logging; I log
>some things which are interested especially for me, but it will be
>logged if logging is activated. One user persisted on the
>meaning that the logged information was fatal for IzPack.
>
>Therefore it will be nice if the new logging stuff will be support
>different topics (as you have decribed) and logs a prefix. May be
>configurable what topic should be logged.
>
>Cheers
>
>Klaus
>
>_______________________________________________
>izpack-devel mailing list
>izpack-devel at lists.berlios.de
>https://lists.berlios.de/mailman/listinfo/izpack-devel
>

_______________________________________________
izpack-devel mailing list
izpack-devel at lists.berlios.de
https://lists.berlios.de/mailman/listinfo/izpack-devel


-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/ms-tnef
Size: 11740 bytes
Desc: not available
Url : https://lists.berlios.de/pipermail/izpack-devel/attachments/20061201/20d61396/attachment.bin 


More information about the izpack-devel mailing list