Hey Max,

2011/7/11 Max Mikhanosha <[email protected]>:
> WT has its own custom logging system system, which is not very
> convenient. This can be seen by the fact that there are plenty of
> commented or ifdefed log statements logging to std::cerr.

As if you needed evidence :-)

> I'm attaching a patch adding log4cplus support for WT. Its done in
> un-obtrusive way so that if log4cplus support is not enabled, existing
> functionality is preserved. No code is removed, instead ifdefs are
> used.

I like the idea. Indeed so far we were inserting (and also removing)
logging in the library when tracking down bugs and a better approach
is certainly welcome.

The one thing that the WApplication::log() API did however, was format
the message into a Common LogFile Format (like accesslog logging). I
guess this feature is lost ? But I am not sure whether this was
actually useful... The only use for Commons LogFile Format is
statistics from files such as the access log...

> Patch is attached below:

Appreciated, I will take a look as soon as possible.

One issue that we will need to consider is that the Apache License
(v2) actually is not compatible with GNU License v2 ... This may
provide the incentive to move to GPL v3, something which has been
discussed in the past already.

http://www.apache.org/licenses/GPL-compatibility.html

Regards,
koen

------------------------------------------------------------------------------
All of the data generated in your IT infrastructure is seriously valuable.
Why? It contains a definitive record of application performance, security 
threats, fraudulent activity, and more. Splunk takes this data and makes 
sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-d2d-c2
_______________________________________________
witty-interest mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/witty-interest

Reply via email to