Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to jaxen 1.2.0 #186

Closed
trancexpress opened this issue Aug 3, 2020 · 4 comments
Closed

Update to jaxen 1.2.0 #186

trancexpress opened this issue Aug 3, 2020 · 4 comments

Comments

@trancexpress
Copy link

My team is in the process of updating an Eclipse-based product to compile with Java 11. Among other problems, we run into split package problems with jdom 2.0.4:

[javac] The package org.w3c.dom is accessible from more than one module: <unnamed>, java.xml

This seems to be due to version of jaxen used in the jdom bundle; the jaxen version has the class org.w3c.dom.UserDataHandler, causing the compile error. We would like to avoid use of --limit-modules and compile cleanly; we checked the latest available jdom 2.0.6 but it contains jaxen 1.1.6 (up from jaxen 1.1.4 in our case). Unfortunately this jaxen version still contains a org.w3c.dom package.

jaxen 1.2.0 on the other hand does not. So it would be great if a jdom bundle is available with jaxen 1.2.0, to support applications that compile against JDK 9+.

@hunterhacker
Copy link
Owner

Simeon, do you want to take the lead on a release? Rolf had been doing it but he's gone silent (not sure why, hoping for the best).

@trancexpress
Copy link
Author

trancexpress commented Aug 4, 2020

I've consulted my team, we'll try to separate the jaxen dependency in the current jdom release we use. So that we don't compile against jaxen (which we don't need).

Our hands are unfortunately full already, we cannot devote time to preparing a jdom release.

@rolfl
Copy link
Collaborator

rolfl commented Jul 2, 2021

And, I have resurfaced. Apologies, I was inadvertently sending all my JDOM correspondence in to an unmonitored mail folder.... This issue will take some finangling to get the builds through. There's a fair backlog I need to address, this will be part of it.

@rolfl rolfl closed this as completed in 25cf24e Jul 2, 2021
rolfl added a commit that referenced this issue Jul 2, 2021
@trancexpress
Copy link
Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants