On Apr 25, 2007, at 10:50 , Grégory Joseph wrote:


On Apr 24, 2007, at 23:55 , [EMAIL PROTECTED] wrote:

ewe also need a slf4j implementation, or magnolia will not run... if you want to remove it from the default dependencies you should make it a test dependency for modules and add it in the webapp
Woops, somehow I was sure it was there. As a matter of fact, it already ends up in the webapp without this commit, but I'm not sure how.
I'd be in favor of having on the webapp rather than core ?
+ now that I look at this, core should have an explicit dependency on log4j, since it uses its api. I just looked at the dependencies report: slf4j-log4j is brought by jackrabbit, and log4j itself is brought by openwfe.

Ha ha! I see now, since you upgrade to jackrabbit1.3, slf4j-log4j is not brought along anymore ;)

I'd still rather put the slf4j-log4j dep on the webapp instead of core, wdyt?

Cheers

g


----------------------------------------------------------------
for list details see
http://www.magnolia.info/en/developer.html
----------------------------------------------------------------

Reply via email to