[ 
https://issues.apache.org/activemq/browse/SM-1949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58844#action_58844
 ] 

Tatu Saloranta commented on SM-1949:
------------------------------------

Hmmh. Codehaus Jira is now down, so I can't add comments to Woodstox issue.

But this all is very confusing. Firstly, it is 1.0-2,  not 1.0.2; meaning that 
strictly speaking it is not a newer version (as in they probably come from 
different source tree -- 1.0.1 came from stax.codehaus.org, stuff BEA wrote).
Also, I don't know if it actually contains the fix to NPE that 1.0.0 had (I 
hope, but it all depends on where exactly sources came from). The whole STAX 
API is such a cluster.

At any rate, ideally there would just be a range, but since I don't have a good 
way to verify that what I use works, I probably don't want to do that.

In the meantime, maybe ActiveMQ could just override stax api version Woodstox 
claims?
Stax API is included in JDK 1.6, so in most cases it should not even be 
deployed at all, but Maven does not have a way indicate "provided by JDK X.X" 
option. Woodstox itself should work on any 1.0.x version.


> Upgrade stax-api version
> ------------------------
>
>                 Key: SM-1949
>                 URL: https://issues.apache.org/activemq/browse/SM-1949
>             Project: ServiceMix
>          Issue Type: Bug
>            Reporter: Hadrian Zbarcea
>            Assignee: Chris Custine
>             Fix For: 3.4
>
>         Attachments: sm-1949.patch
>
>
> The main pom.xml defines a dependency on stax:stax-api:jar:1.0.1 which:
> 1. is old (Mar 2006)
> 2. defines a javax.xml.XMLConstants class that does not define 
> XMLConstants.NULL_NS_URI and causes compile errors in some ides.
> 3. mvn dependency:tree in servicemix-core shows dependencies on two different 
> version of stax-api (edited for brevity):
> {code}
> INFO] org.apache.servicemix:servicemix-audit:jar:3.4.0-fuse-SNAPSHOT
> [INFO] +- 
> org.apache.servicemix:servicemix-core:jar:3.4.0-fuse-SNAPSHOT:compile
> [INFO] |  +- 
> org.apache.geronimo.modules:geronimo-transaction:jar:2.1.4:compile
> [INFO] |  |  +- org.apache.geronimo.framework:geronimo-core:jar:2.1.4:compile
> [INFO] |  |  |  +- 
> org.apache.geronimo.framework:geronimo-system:jar:2.1.4:compile
> [INFO] |  |  |  |  |  \- javax.xml.stream:stax-api:jar:1.0-2:compile
> [...]
> [INFO] |  +- stax:stax-api:jar:1.0.1:compile
> {code}
> I believe the correct version we should use is 
> javax.xml.stream:stax-api:jar:1.0-2 (the groupId has changed as well). I 
> tested the attached patch and everything sees in order. I hope I didn't miss 
> anything.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
https://issues.apache.org/activemq/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to