Skip to content

Bump Java build requirements to 17, 21 & 25#167

Open
sbesson wants to merge 5 commits intoglencoesoftware:masterfrom
sbesson:gradle9_java25
Open

Bump Java build requirements to 17, 21 & 25#167
sbesson wants to merge 5 commits intoglencoesoftware:masterfrom
sbesson:gradle9_java25

Conversation

@sbesson
Copy link
Copy Markdown
Member

@sbesson sbesson commented Feb 26, 2026

Consistently with the Java support roadmap discussed at the weekly OME meeting - https://docs.google.com/spreadsheets/d/1hXjl3WXM5iXHVZbCi4CI-FWF_rcnPOdhlWig48Cub5c/edit?gid=1511932992#gid=1511932992, this PR updates the micro-service to:

@sbesson sbesson requested review from chris-allan and kkoz February 26, 2026 14:02
@sbesson sbesson changed the title Bump Java build requirements (17+ runtime) Bump Java build requirements to 17, 21 & 25 Feb 26, 2026
@chris-allan
Copy link
Copy Markdown
Member

Thanks, @sbesson. I was already using a version of this locally since my Eclipse driven workflow as currently Java 25. Will take a closer look at the details but I'm definitely fine with it conceptually.

@chris-allan
Copy link
Copy Markdown
Member

Might be an idea to loose the Gradle 10 warnings by performing these changes?

diff --git a/build.gradle b/build.gradle
index aa1672a..de1aebb 100644
--- a/build.gradle
+++ b/build.gradle
@@ -34,23 +34,23 @@ repositories {
     mavenCentral()
     mavenLocal()
     maven {
-        name 'Unidata'
-        url 'https://artifacts.glencoesoftware.com/artifactory/unidata-releases'
+        name = 'Unidata'
+        url = 'https://artifacts.glencoesoftware.com/artifactory/unidata-releases'
     }
     maven {
-        url 'https://artifacts.glencoesoftware.com/artifactory/ome.releases'
+        url = 'https://artifacts.glencoesoftware.com/artifactory/ome.releases'
     }
     maven {
-        url 'https://artifacts.glencoesoftware.com/artifactory/ome.external'
+        url = 'https://artifacts.glencoesoftware.com/artifactory/ome.external'
     }
     maven {
-        url 'https://artifacts.glencoesoftware.com/artifactory/gs-omero-snapshots-local'
+        url = 'https://artifacts.glencoesoftware.com/artifactory/gs-omero-snapshots-local'
     }
     maven {
-        url 'https://repo.glencoesoftware.com/repository/bioformats2raw2ometiff/'
+        url = 'https://repo.glencoesoftware.com/repository/bioformats2raw2ometiff/'
     }
     maven {
-        url 'https://artifacts.glencoesoftware.com/artifactory/scijava-thirdparty'
+        url = 'https://artifacts.glencoesoftware.com/artifactory/scijava-thirdparty'
     }
 }

Copy link
Copy Markdown
Member

@chris-allan chris-allan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Builds nicely for me on 17 (Linux) and 25 (Linux and Windows). Code also still runs nicely on 11 (Linux) built with 25 as well as 17 (Linux) and 25 (Windows).

Copy link
Copy Markdown
Member

@kkoz kkoz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Build fails with java 11, succeeds with the others.

Ran 11, 17, 21, and 25 against a java 25 build successfully (Ubuntu 22.04)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants