On Sun, 30 Dec 2001, Sylvain Wallez wrote:

> giacomo wrote:
>
> <snip/>
>
>
> >>>>The only drawback is that LogKit complains about categories not
> >>>>explicitly declared in logkit.xconf. Maybe LogKit shouldn't be so picky
> >>>>about that and just act as a configuration front-end to Hierarchy (i.e.
> >>>>remove the m_loggers HashMap).
> >>>>
> >>>This can be fixed (I wrote it) but IIRC it's a debugging (or should
> >>>be) message and thus might be helpfull in that aspect.
> >>>
> >>I would be thankful if you fix it, because it produces a _warning_
> >>message each time a component asks for a logger that's not in the
> >>configuration-defined categories.
> >>
> >
> > Ok, I've reduced from WARN to DEBUG and will commit it soon into C2.
> >
> > Giacomo
> >
>
> Thanks, Giacomo, but I was saying that IMHO, LogKitManager shouldn't
> even care about that. It's role is to configure a Hierarchy (most often
> its top-level categories) and give access to this hierarchy to get
> loggers, but the definition of categories is the responsibility of
> xconf/sitemap writers.

Right, and thus debugging messages are quite usefull for that.

> With the hierarchized categories we will now have (many +1 for this
> vote), LogKitManager will issue a great number of debug messages which
> don't give any usefull information.

I don't agree about "don't give any usefull information". The
information is that there could be other categories/hierarchies to
possibly taken care of with a different logkit configuration. And, hey,
they're "only" debugging messages. Have you ever run a SAMBA server or a
Squid proxy in debugging mode? Do you think that information is usefull
for you? Probably not, because you are not used to that kind of
information. But is common to debugging messages that not everybody is
able to understand. So my proposal is to move to INFO level instead.

Giacomo

>
> What do you think ?
>
> Sylvain.
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, email: [EMAIL PROTECTED]
>
>
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, email: [EMAIL PROTECTED]

Reply via email to