Skip to content

Commit

Permalink
Bump SecureJarHandler after breaking changes (#125)
Browse files Browse the repository at this point in the history
* Bump SecureJarHandler

* Bump SJH

* Bump to run on Java 21, since SJH requires it.

* Bump to run on Java 21, since SJH requires it.

* Bump to run on Java 21, since SJH requires it.

* Add add-opens for JMH
  • Loading branch information
shartte authored May 1, 2024
1 parent e86cd31 commit 4612c65
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 11 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build-prs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ jobs:
build:
uses: neoforged/actions/.github/workflows/build-prs.yml@main
with:
java: 17
java: 21
gradle_tasks: test
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
release:
uses: neoforged/actions/.github/workflows/gradle-publish.yml@main
with:
java: 17
java: 21
pre_gradle_tasks: test
gradle_tasks: publish
secrets:
Expand Down
11 changes: 5 additions & 6 deletions .github/workflows/test_jvms.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,13 @@ jobs:
max-parallel: 10
fail-fast: false
matrix:
jvm_version: [ 17, 18, 19, 20 ]
jdk: [ Graal_VM, Adoptium, Azul, IBM, Oracle, Amazon, BellSoft, SAP ]
exclude:
- jdk: Graal_VM
jvm_version: 18 # Graal doesn't have a dedicated J18 version
jvm_version: [ 21, 22 ]
jdk: [ Graal_VM, Adoptium, Azul, Oracle, Amazon, BellSoft, SAP ]
include:
- jdk: IBM
jvm_version: 21 # IBM only has 21
- jdk: Microsoft
jvm_version: 17 # MS OpenJDK only has 11 and 177
jvm_version: 21 # MS OpenJDK only has 21
steps:
- name: Checkout repository
uses: actions/checkout@main
Expand Down
5 changes: 3 additions & 2 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ plugins {
}

java {
toolchain.languageVersion = JavaLanguageVersion.of(17)
toolchain.languageVersion = JavaLanguageVersion.of(21)
withSourcesJar()
}

Expand Down Expand Up @@ -115,6 +115,7 @@ tasks.register('jmh', JavaExec) {
jvmArgs(
'--module-path', sourceSets.jmh.runtimeClasspath.asPath,
'--add-modules', 'ALL-MODULE-PATH',
'--add-opens', 'java.base/java.lang.invoke=cpw.mods.securejarhandler',
'--add-opens', 'cpw.mods.modlauncher/cpw.mods.modlauncher=powermock.reflect',
'--add-opens', 'cpw.mods.modlauncher/cpw.mods.modlauncher=cpw.mods.modlauncher.benchmarks',
'--add-exports', 'cpw.mods.modlauncher.benchmarks/cpw.mods.modlauncher.benchmarks.jmh_generated=jmh.core'
Expand Down Expand Up @@ -189,7 +190,7 @@ publishing {
from components.java
pom {
name = 'Mod Launcher'
description = 'Java 17 mod launcher'
description = 'Java 21 mod launcher'
pomUtils.githubRepo(it, 'modlauncher', 'McModLauncher')

developers {
Expand Down
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#org.gradle.java.home=/home/cpw/minecraft/runtime/java-runtime-gamma/linux/java-runtime-gamma

asm_version=9.5
sjh_version=2.1.11
sjh_version=3.0.4
log4j_version=2.20.0
jopt_version=5.0.4
jetbrains_ann_version=24.0.1
Expand Down

0 comments on commit 4612c65

Please sign in to comment.