logging-log4j-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Scott Deboy (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (LOG4J2-238) OSGi dependency failures in core
Date Sat, 25 May 2013 09:28:22 GMT

    [ https://issues.apache.org/jira/browse/LOG4J2-238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13667035#comment-13667035
] 

Scott Deboy commented on LOG4J2-238:
------------------------------------

Would you mind removing the commented out code?
                
> OSGi dependency failures in core
> --------------------------------
>
>                 Key: LOG4J2-238
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-238
>             Project: Log4j 2
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 2.0-beta5
>         Environment: OSGi (Eclipse 4.2.2, but I think the issue is generic OSGi)
>            Reporter: Bob Kerns
>              Labels: OSGi
>         Attachments: 0001-LOG4J2-238-and-Log472-159.-OSGi-coordination-of-API-.patch
>
>
> To get the core module to load, in addition to re-fixing what was almost fixed in LOG4J2-159
I had to make several packages optional.
> * com.lmax.disruptor
> * com.lmax.disruptor.dsl
> * com.lmax.disruptor.util
> These provided a dependency on sun.misc, which I could hack around to make available
but isn't normally available in OSGi and thus not a dependency I can easily put into a product.
I believe the dependency is on sun.misc.Unsafe  I'd like to use it...
> There is also a direct dependency somewhere on
> * sun.misc (also Unsafe)
> * com.sun.tools.jconsole -- I think this unlikely to be used in an OSGi environment,
so optional is appropriate.
> * org.codehaus.jackson
> * org.codehaus.jackson.map
> These would be easy enough to satisfy, but since most people won't need JSON logging,
the dependency should be optional.
> I think the correct minimal fix is just to make them all optional in the manifest. Eliminating
the need for sun.misc would be a good further step

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org


Mime
View raw message