jenkinsci / maven-hpi-plugin

Maven plugin for building Jenkins plugins
http://jenkinsci.github.io/maven-hpi-plugin/
Apache License 2.0
73 stars 87 forks source link

Require Apache Maven 3.9.6 or later #668

Closed Vlatombe closed 1 month ago

Vlatombe commented 1 month ago

Previous changes in Maven require 3.9.6 or later for Java 17 compatibility. Fixes #667

See also

Testing done

Submitter checklist

basil commented 1 month ago

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 incrementals-tools.

batmat commented 1 month ago

Should this change not be released (would be 3.59 I assume)? So we can propagate this requirement around, e.g. on https://github.com/jenkinsci/plugin-pom/blob/5485e031ff8f6de66291f02626cd959aba62320f/pom.xml#L77 and https://github.com/jenkinsci/pom/blob/66f86b9fad7fc5112c21055017d40568ce294e15/pom.xml#L98 ?

Vlatombe commented 1 month ago

I don't think it is that urgent, because of https://github.com/jenkinsci/pom/pull/612 & https://github.com/jenkinsci/plugin-pom/pull/1018 doing the same check essentially earlier in the build lifecycle. Call it defense in depth. Would be only useful if a plugin decides to pick up the latest maven hpi plugin without bumping the parent POM.

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.