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 maven-invoker-plugin to 3.2.2 to fix JDK16 build #448

Merged
merged 1 commit into from Feb 25, 2021

Conversation

famod
Copy link
Contributor

@famod famod commented Feb 21, 2021

Fixes these problems in the JDK16 build:

[INFO] run post-build script verify.bsh
[INFO]   The post-build script did not succeed. Sourced file: inline evaluation of: ``import java.io.*; import java.util.regex.*;  try {     File file = new File( bas . . . '' unknown error: Unable to make public java.lang.AbstractStringBuilder java.lang.AbstractStringBuilder.append(java.lang.String) accessible: module java.base does not "opens java.lang" to unnamed module @430c04a6
[INFO]           it-update-child-modules-002/pom.xml .............. FAILED (6.3 s)

This should be merged ASAP since all more or less recent PRs are affected by this problem (and master of course). /cc @sparsick @mfriedenhagen

I guess sooner or later dependabot would have created a PR for this update, but given the importance I thought better do it now.

@sparsick
Copy link
Contributor

@famod I have no permission to merge your PR. But maybe @khmarbaise can help here. BTW, it is a good idea 👍

@mfriedenhagen mfriedenhagen merged commit cee63d8 into mojohaus:master Feb 25, 2021
@mfriedenhagen
Copy link
Member

Thanks, @famod

@famod famod deleted the invoker-update branch February 25, 2021 18:43
@stefanseifert stefanseifert added the dependencies Pull requests that update a dependency file label Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants