Skip to content

Commit 1d877ac

Browse files
committed
Set Java version to 21 for pombast melt
This lets the melt action pass -- for now. A comment explains why this is tenuous, though.
1 parent f74416b commit 1d877ac

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

pombast.toml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,13 @@ footer = "footer.html"
1313
nexus-base = "https://maven.scijava.org"
1414

1515
[melt]
16-
java-version = 11
16+
# Some managed components require Java 21+. So it makes sense to melt with that version of Java.
17+
# Unfortunately, the scijava.jvm.version is currently set to 11 by default.
18+
# So even though OpenJDK 21 is being used to run Maven, the enforcer plugin still
19+
# complains when encountering any Java libraries with bytecode newer than 11.
20+
# Setting scijava.jvm.version to 21 by default would avoid the issue, but...
21+
# is that really a good idea? Setting to 11 seems like a nice baseline compromise right now.
22+
java-version = 21 # Maximum Java version of all included component + deps.
1723
template = "tests/mega-melt-template.xml"
1824
excludes = [
1925
# TEMP: Exclude org.bytedeco:hdf5 until cisd:jhdf5 is gone.

0 commit comments

Comments
 (0)