Am Mittwoch, 25. April 2007 schrieb Grégory Joseph:
> 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?

+1  - and one implementation (slf4j-log4j) for scope "test".

O.



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

Reply via email to