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

upgraded version.compiler.plugin #52

Merged
merged 1 commit into from
Nov 14, 2018
Merged

Conversation

mbiarnes
Copy link
Contributor

@mbiarnes mbiarnes commented Oct 3, 2018

justification: see https://blog.codefx.org/java/java-11-migration-guide/
JDK11 build friendly

@ge0ffrey
Copy link

@kabir For OpenJDK 11 build compatibility, we need this PR merged and released. Would you mind taking a look?

"Here are the recommended minimum versions for a few tools:
Maven: 3.5.0
compiler plugin: 3.8.0
surefire and failsafe: 2.22.0"
Source: https://blog.codefx.org/java/java-11-migration-guide/#Update-All-The-Things

@dmlloyd
Copy link
Member

dmlloyd commented Nov 14, 2018

This will now require a 3.8.0-jboss-1 tag for maven-compiler-plugin which has the additionalClassPathItems patch.

@dmlloyd
Copy link
Member

dmlloyd commented Nov 14, 2018

The 3.8.0-jboss-1 tag is released to our Nexus instance. Now the PR just has to be updated; I'll go ahead and do that.

@dmlloyd dmlloyd merged commit d06b112 into jboss:master Nov 14, 2018
@dmlloyd
Copy link
Member

dmlloyd commented Nov 14, 2018

All set!

@bartoszmajsak
Copy link
Contributor

Any chance to contribute it back to the official maven compiler?

@dmlloyd
Copy link
Member

dmlloyd commented Nov 21, 2018

@ge0ffrey
Copy link

@dmlloyd Thanks for picking this up David!

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

Successfully merging this pull request may close these issues.

4 participants