-
Notifications
You must be signed in to change notification settings - Fork 129
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
checkstyle-tester is not Java 11 compatible #352
Comments
@Vampire is that the full stack trace and error? |
yes, it is the full stack trace |
Just trying to determine where jaxb is located to see if it can be removed. I don't see it listed directly which means some plugin we are using in the POM must be calling it, unless somehow groovy is requesting it. I will have to install Java 11 and see if I can find anything more. |
I saw this when testing it out:
The maven plugin is an integrate part of our regression until Issue #273 is completed. Our main repo |
I cannot reproduce the error locally with windows and java 11.
|
Why do you think it has anything to do with maven-checkstyle or any maven run at all? |
It also depends on the Groovy version. |
You say
According to the stack trace, it only tells me the exception occurred when diff.groovy is calling launch.groovy.
Yes and I got no errors/exceptions.
I am using the same version.
I saw this and just assumed groovy was trying to do something extra when it saw it had to load the jaxb class. |
Yes, and that instantly, no maven output or anything.
Nope, still failing when I use Java 11.
I added
|
Hm, seems to be a Groovy problem if anything. |
Hm, in the |
in |
Hm, strange, because if I do either of first or last of the three points in the bottom of http://groovy-lang.org/releasenotes/groovy-2.5.html#Groovy2.5releasenotes-Addendum251 it works (either removing groovy-jaxb or adding the other four to the lib directory) |
I really wonder that it works for you. |
|
My quotes and escaping is for Bash, not for PS |
wth, from cmd.exe I even get the exception with |
https://issues.apache.org/jira/browse/GROOVY-9039 if anyone is interested. |
When launching the checkstyle tester with
JAVA_HOME
set to Java 11, it starts until it has built the base branch and then it aborts with this exception:javax.xml.bind.JAXBContext
like all other JAXB stuff was removed from Java 11 in favor of using explicit dependencies.The text was updated successfully, but these errors were encountered: