slachiewicz commented on PR #501: URL: https://github.com/apache/maven-resources-plugin/pull/501#issuecomment-5178607544
Confirmed in production — this repo's own master build #171: ``` OS: linux JDK: 17 Maven: 4.0.x => Label: ubuntu JDK: jdk_17_latest Maven: maven_4_latest OS: linux JDK: 21 Maven: 4.0.x => Label: ubuntu JDK: jdk_21_latest Maven: maven_4_latest OS: linux JDK: 17 Maven: [4.0.x] => Label: ubuntu JDK: jdk_17_latest Maven: maven_3_latest ``` The first two lines are the `maven:` list, correctly iterated to a String. The third is `siteMvn` passed whole, stringified to `[4.0.x]`, resolved to `maven_3_latest` — the site stage running on Maven 3, exactly as this PR describes. The workspace directory is even named `linux-jdk17-m[4.0.x]_site`. Since the shared library started provisioning Maven through the wrapper, the same value now also reaches `-Dmaven=`, so it fails outright rather than silently downgrading: ``` + mvn ... wrapper -Dmaven=[4.0.x] wget: Failed to fetch .../apache-maven-[4.0.x]-bin.zip Failed in branch linux-jdk17-m[4.0.x]_site ``` The library-side fixes are apache/maven-jenkins-env#8 and apache/maven-jenkins-lib#23. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
