Skip to content

[POC] Migrate to Java 21 - code #2281

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

Pankraz76
Copy link
Contributor

@Pankraz76 Pankraz76 commented May 1, 2025

@Pankraz76 Pankraz76 changed the title Pull #16981: Migrate to Java 21 Pull #2281: Migrate to Java 21 May 1, 2025
@Pankraz76
Copy link
Contributor Author

Pankraz76 commented May 1, 2025

Items:

  • rework @Disabled("tmp-fix")

  • image
  • image
  • [INFO] BUILD SUCCESS

  • image

@Pankraz76
Copy link
Contributor Author

is this any good?

@desruisseaux
Copy link
Contributor

Does it needs to be synchronized with #2277 for avoiding overlaps and therefore, potential merge conflicts?

@Pankraz76
Copy link
Contributor Author

good catch, thx.

@Bukama
Copy link

Bukama commented May 1, 2025

Rising the Maven version requires an official, succeeded vote.

@Pankraz76
Copy link
Contributor Author

Rising the Maven version requires an official, succeeded vote.

is there an template? I wanted to pitch an similar Issue, buts its not allowed:

Copy link
Contributor

@gnodet gnodet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need a separate PR for bumping the required JDK to 21.
Other PRs can follow and be based on that one.

Copy link
Contributor

@gnodet gnodet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need a separate PR for bumping the required JDK to 21.
Other PRs can follow and be based on that one.

@Pankraz76 Pankraz76 changed the title Pull #2281: Migrate to Java 21 Pull #2281: Migrate to Java 21 May 9, 2025
@Pankraz76 Pankraz76 marked this pull request as draft May 9, 2025 11:23
@Pankraz76 Pankraz76 changed the title Pull #2281: Migrate to Java 21 [POC] Migrate to Java 21 - code May 10, 2025
@Pankraz76
Copy link
Contributor Author

We need a separate PR for bumping the required JDK to 21. Other PRs can follow and be based on that one.

@Pankraz76
Copy link
Contributor Author

Pankraz76 commented May 10, 2025

item: ResourceNotFoundException: Could not find resource 'licenses/unrecognized.txt'. at remote-resources

image
[INFO] --- remote-resources:3.3.0:process (process-resource-bundles) @ apache-maven ---
[INFO] Preparing remote bundle org.apache.apache.resources:apache-jar-resource-bundle:1.7
[INFO] Copying 3 resources from 1 bundle.
[INFO] Filtering appended resource: META-INF/LICENSE.vm
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  2.027 s
[INFO] Finished at: 2025-05-11T00:46:30+02:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-remote-resources-plugin:3.3.0:process (process-resource-bundles) on project apache-maven: Error rendering Velocity resource 'META-INF/LICENSE.vm': Invocation of method 'getResourceAsFile' in  class org.codehaus.plexus.resource.DefaultResourceManager threw exception org.codehaus.plexus.resource.loader.ResourceNotFoundException: Could not find resource 'licenses/unrecognized.txt'. at remote-resources[line 75, column 39] -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
➜  apache-maven git:(code-infra--Migrate-to-Java-21) 

@Pankraz76 Pankraz76 requested a review from gnodet May 10, 2025 22:53
@Pankraz76 Pankraz76 changed the title [POC] Migrate to Java 21 - code [POC] Migrate to Java 21 - code May 10, 2025
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