Include one ore more SLF4J "adapters"

Description

Hibernate, as of 3.3, no longer distributes all jars needed to run the framework. SLF4J requires the inclusion of the SLF4J library as well as either a native or a wrapped implementation. The manual vaguely references such a need but it's never quite clear that the user must visit the SLF4J website and obtain these jars himself. Maven users are of course unaffected since the additional SLF4J jars are published in the central repo.

"Adapter" may be the wrong term here but I think the point has been made

Environment

None

Activity

Show:
Steve Ebersole
November 16, 2009, 7:30 AM

Ok, this is really my last comment on this issue. Far too much time on it already...

Its not "a jar or two". I am not sure which all strictly constitute bindings but I see what appears to be 7, and that is not even accounting for dependencies of the bindings! For example, if we bundle slf4j-log4j12 by y'alls logic we also need to bundle some 1.2.x version of log4j, etc.

Andrew C. Oliver
November 16, 2009, 7:43 AM

My issue is that you still have the libel above that I was some contractor. http://www.secinfo.com/d14D5a.v43sb.htm

Steve Ebersole
November 16, 2009, 7:54 AM

You are correct it was when you moved into a "consultant" role, not "contractor" role.

Ceki Gulcu
November 16, 2009, 10:08 AM

I reiterate my offer to default to an nop implementation in case no slf4j-binding could be found on the class path. It's a small change in the slf4j-api which would imo nicely address this issue. If interested, please open an slf4j bug report.

Andrew C. Oliver
November 16, 2009, 3:38 PM

Done: http://bugzilla.slf4j.org/show_bug.cgi?id=159

(sorry Ceki if I got the details wrong. I am not all that familiar with this particular library)

Rejected

Assignee

Steve Ebersole

Reporter

Chris Bredesen

Fix versions

None

Labels

None

backPortable

None

Suitable for new contributors

None

Requires Release Note

None

Pull Request

None

backportDecision

None

Components

Affects versions

Priority

Major
Configure