-
Notifications
You must be signed in to change notification settings - Fork 88
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
Require Apache Maven 3.9.6 or later #668
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! We've been running Apache Maven 3.9.9 and its predecessor versions on ci.jenkins.io and its friends (like release.ci.jenkins.io) since those versions were released. No issues detected as far as I recall.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! There are a few other places in the ecosystem where we have a 3.8.1 baseline, such as acceptance-test-harness
and other Jenkins Maven plugins (e.g., access-modifier-checker
). I will try to hunt those down and update them as well.
I moved some of the rest of the ecosystem to Maven 3.9.6 as well. I did not have the time to look at |
Should this change not be released (would be 3.59 I assume)? |
I don't think it is that urgent, because of jenkinsci/pom#612 & jenkinsci/plugin-pom#1018 doing the same check essentially earlier in the build lifecycle. However, this makes me think that we could have found out about this issue earlier, if we had a build of maven-hpi-plugin using the minimum maven version that is specified in the build. Under this assumption, I think the plugin ITs would have failed in such an environment. |
Previous changes in Maven require 3.9.6 or later for Java 17 compatibility. Fixes #667
See also
Testing done
Submitter checklist