On Aug 14, 2008, at 9:17 AM, <[EMAIL PROTECTED]> <[EMAIL PROTECTED]
> wrote:
Hi Curt,
There is also an encapsulation of log4j in felix trunk/commons/
log4j . If I understand well, the proposal is that the log4j
artifact in apache log4j project is also a bundle ? great ! (even
better if this bundle is made available on an "OBR" reopsitory).
The proposal is that the log4j-1.2.16.jar distributed in the upcoming
Apache log4j 1.2.16 release is a bundle. Don't know what an OBR
repository is. On previous versions we have published to the Maven 2
master repository. Guess I'll look it up unless you want to provide
references.
Concerning optional packages, maybe you can :
remove almost everything from "Import-Package"
and add a "DynamicImport-Package" clause so that if specific
appenders are required by configuration, and a bundle provides the
packages needed by these appenders, they can be used, but if you
need only FileAppender and ConsoleAppender you can use a very
minimal configuration?
I read the section on DynamicImport in the R3 draft and it didn't seem
to fit with the scenario since it was talking about classes that were
loaded using Class.forName() or similar. The ideal answer would be to
split log4j.jar into a code and a fragment that depended on JMS, a
fragment that depended on JDBC, etc, but that isn't in the cards for
the mainstream release.
I have personnaly recently built a log4j 1.2.13 bundle, starting
from the felix log4j project which was providing 1.2.12, and I
suppressed the importation of javax.* packages in order to limit the
number of dependencies. Seems to work anyway.
Anne
I'd guess that almost all of the javax packages would be provided by
the underlying environment and there'd be very little overhead in
resolving them. Though that is just a wild guess based on no
experience. Would you suggest adding a "!java.*, !javax.*" to the
imports? If so, why doesn't maven-felix-plugin do that by default?
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
