Revert "Bump org.jenkins-ci:jenkins from 1.122 to 1.123"

[jglick]

Reverts #844

Currently stumped by a JTH release failure. https://github.com/jenkinsci/jenkins-test-harness/actions/runs/11060243118/job/30730386885 complains about missing org.jenkins-ci.main:jenkins-war:executable-war:2.476. Indeed https://repo.jenkins-ci.org/public/org/jenkins-ci/main/jenkins-war/2.476/ only has jenkins-war-2.476.war but that is by design; jenkinsci/maven-hpi-plugin#65 + #61 are supposed to look up the *.war and just add it to the Java classpath. https://github.com/jenkinsci/jenkins-test-harness/actions/runs/10966208526/job/30453668284 worked normally just a few days ago (using the same versions of Maven and Java), and the only POM change since then is #844 which… hmm, https://github.com/jenkinsci/pom/releases/tag/jenkins-1.123 does not mention it but jenkinsci/pom@jenkins-1.122...jenkins-1.123 does show this including jenkinsci/pom#608 with a lot of changes in https://github.com/jenkinsci/maven-hpi-plugin/releases/tag/maven-hpi-plugin-3.58 like jenkinsci/maven-hpi-plugin#649 which might be causing trouble somehow?

I failed to reproduce any problem locally with

docker run -ti --rm --name mvn -u $(id -u):$(id -g) -v "$PWD":/usr/src/mymaven -w /usr/src/mymaven maven:3-eclipse-temurin-17 mvn -Duser.home=/tmp clean compile

[jglick]

FWIW a similar error can reproduced from

mvn verify -U -e -ntp

when suppressing the resource which is supposed to make it work:

diff --git pom.xml pom.xml
index 47be48f8c8..a1618e87ae 100644
--- pom.xml
+++ pom.xml
@@ -331,7 +331,6 @@ THE SOFTWARE.
       <plugin>
         <groupId>org.jenkins-ci.tools</groupId>
         <artifactId>maven-hpi-plugin</artifactId>
-        <extensions>true</extensions>
       </plugin>
     </plugins>
   </build>

[jglick]

https://github.com/actions/setup-java/releases/tag/v4.4.0 (2 days ago) has a few changes but nothing obviously related (and as noted, the Maven version did not seem to change; see actions/setup-java#685).

[jglick]

Aha! Reproduces with

docker run -ti --rm --name mvn -u $(id -u):$(id -g) -v "$PWD":/usr/src/mymaven -w /usr/src/mymaven maven:3.8.8-eclipse-temurin-17 mvn -Duser.home=/tmp clean compile

I will work on bisection.

[jglick]

https://github.com/jenkinsci/jenkins-test-harness/releases/tag/2299.v7e8d22e797d3 succeeded. Bisection points to jenkinsci/maven-hpi-plugin#628 as the culprit. I suspect the switch to Java 17 bytecode is responsible, as in jenkinsci/maven-hpi-plugin#667.