Skip to content

Simplify and generalize TornadoVM version across JDK profiles in pom.xml #99

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.

Sign up for GitHub

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

Merged
mikepapadim merged 1 commit into from
Mar 11, 2026
+9 −9
Merged

Simplify and generalize TornadoVM version across JDK profiles in pom.xml #99

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,10 @@

<properties>
<!-- CI-friendly version: resolved by flatten-maven-plugin at build time -->
<revision>0.4.0</revision>
<jdk.version.suffix></jdk.version.suffix> <!-- empty=JDK21, -jdk25=JDK25 -->
<!-- TornadoVM version: overridden per JDK profile -->
<tornadovm.version>3.0.0</tornadovm.version> <!-- JDK21 default -->
<revision>0.4.0-dev</revision>
<tornadovm.base.version>3.0.0</tornadovm.base.version> <!-- TornadoVM base version (without jdk suffix) -->
<jdk.version.suffix></jdk.version.suffix> <!-- JDK Suffix: empty=JDK21, -jdk25=JDK25 -->
<tornadovm.version>${tornadovm.base.version}</tornadovm.version> <!-- Default: JDK21 (no suffix) -->
<!-- Compiler defaults (overridden by JDK profiles below) -->
<maven.compiler.source>21</maven.compiler.source>
<maven.compiler.target>21</maven.compiler.target>
Expand Down Expand Up @@ -137,7 +137,7 @@
<!-- ─── JDK 21 ───────────────────────────────────────────────────────────
Auto-activates for JDK 21.x builds.
Publishes: gpu-llama3:${revision} (no suffix)
TornadoVM: 3.0.0
TornadoVM: ${tornadovm.base.version}
Adds add-modules jdk.incubator.vector (still incubating in JDK21)
─────────────────────────────────────────────────────────────────────── -->
<profile>
Expand All @@ -146,8 +146,8 @@
<properties>
<maven.compiler.source>21</maven.compiler.source>
<maven.compiler.target>21</maven.compiler.target>
<tornadovm.version>3.0.0</tornadovm.version>
<jdk.version.suffix></jdk.version.suffix>
<!-- JDK21 default - no jdk suffix needed -->
<tornadovm.version>${tornadovm.base.version}</tornadovm.version>
</properties>
<build>
<plugins>
Expand All @@ -169,7 +169,7 @@
<!-- ─── JDK 25 ───────────────────────────────────────────────────────────
Auto-activates for JDK 25.0.2+ builds (minimum required version).
Publishes: gpu-llama3:${revision}-jdk25
TornadoVM: 3.0.0-jdk25
TornadoVM: ${tornadovm.base.version}-jdk25
Vector API is still incubating in JDK 25 (JEP 508 — 10th Incubator);
add-modules jdk.incubator.vector is required for compilation.
─────────────────────────────────────────────────────────────────────── -->
Expand All @@ -179,8 +179,8 @@
<properties>
<maven.compiler.source>25</maven.compiler.source>
<maven.compiler.target>25</maven.compiler.target>
<tornadovm.version>3.0.0-jdk25</tornadovm.version>
<jdk.version.suffix>-jdk25</jdk.version.suffix>
<tornadovm.version>${tornadovm.base.version}${jdk.version.suffix}</tornadovm.version>
</properties>
<build>
<plugins>
Expand Down
Loading