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
[JENKINS-68147] Support Java 17 #75
Conversation
Any chance to be merged? |
FWIW this still doesn't work with the version coming from a macos node:
|
pom.xml
Outdated
<dependency> | ||
<groupId>org.jenkins-ci</groupId> | ||
<artifactId>version-number</artifactId> | ||
<version>1.10</version> | ||
</dependency> |
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.
This is already included in Jenkins core as of 2.342, so rather than duplicate the library in this plugin, I suggest bumping the core baseline to 2.346 or later.
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.
Something like deleting this new dependency and instead doing:
diff --git a/pom.xml b/pom.xml
index 22f57f5..573bf80 100644
--- a/pom.xml
+++ b/pom.xml
@@ -51,15 +51,15 @@
<properties>
<revision>2.3</revision>
<changelist>-SNAPSHOT</changelist>
- <jenkins.version>2.277.4</jenkins.version>
+ <jenkins.version>2.346.1</jenkins.version>
<gitHubRepo>jenkinsci/${project.artifactId}-plugin</gitHubRepo>
</properties>
<dependencyManagement>
<dependencies>
<dependency>
<groupId>io.jenkins.tools.bom</groupId>
- <artifactId>bom-2.277.x</artifactId>
- <version>984.vb5eaac999a7e</version>
+ <artifactId>bom-2.346.x</artifactId>
+ <version>1382.v7d694476f340</version>
<scope>import</scope>
<type>pom</type>
</dependency>
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.
Implemented in 503ada6
Null pointer exception may still be thrown by the called method, need to preserve functionality.
{"1.8.0", JavaSpecificationVersion.JAVA_6.toReleaseVersion()}, | ||
{"11.0.1", JavaSpecificationVersion.JAVA_11.toReleaseVersion()}, |
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.
Why aren't these toClassVersion
? This parameter represents a class version, not a release version.
src/test/java/hudson/plugin/versioncolumn/JVMVersionComparatorTest.java
Outdated
Show resolved
Hide resolved
Superseded by #88 Thanks! |
JENKINS-68147 Support Java 17
Add support of JDK17 using org.jenkins-ci:version-number library