[
https://issues.apache.org/jira/browse/LOG4J2-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Gary Gregory resolved LOG4J2-1723.
----------------------------------
Resolution: Fixed
Assignee: Gary Gregory
Fix Version/s: 2.8
In Git master, please verify and close.
> Unwanted transitive dependency on geronimo-jms_1.1_spec causes OSGi tests to
> fail
> ---------------------------------------------------------------------------------
>
> Key: LOG4J2-1723
> URL: https://issues.apache.org/jira/browse/LOG4J2-1723
> Project: Log4j 2
> Issue Type: Bug
> Components: Core
> Affects Versions: 2.7
> Reporter: Ludovic HOCHET
> Assignee: Gary Gregory
> Priority: Minor
> Fix For: 2.8
>
> Attachments: patch-log4j2-1723.diff
>
>
> The change from maven-bundle-plugin:2.5.3 to 3.2.0, as part of the study of
> LOG4J2-1637, caused the dependency on package javax.jms to become non
> optional which made the OSGi tests to fails. Post LOG4J2-1637, LOG4J2-1664
> was created to improve the OSGi tests and this problem was "offloaded" to it.
> Since this problem does not originate in the OSGi tests, I'm creating a new
> issue for it.
> Since this appeared with a change of maven-bundle-plugin, I had a look into
> how it processed optional dependencies since it was apparently not handling
> the optional declaration of jboss-jms-api_1.1_spec. As it happened it did add
> the javax.jms package from that dependency to the list of optional packages,
> but at the same time it was adding the same package to the list of required
> packages. Further checks showed that it was picking up the required package
> from geronimo-jms_1.1_spec which is a transitive dependency of
> activemq-broker (shown with dependency:tree). Since the JMS specifications
> are already included with jboss-jms-api_1.1_spec, the geronimo-jms_1.1_spec
> should be excluded.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
