Add JPMS support via Multi-Release Jar (#59)

KotlinCrypto · Mar 18, 2024 · f9ea2dd · f9ea2dd
1 parent e7a5f0d
commit f9ea2dd
Show file tree

Hide file tree

Showing 10 changed files with 62 additions and 10 deletions.
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -42,11 +42,20 @@ jobs:
           key: ${{ runner.os }}-gradle-caches-${{ hashFiles('**/*.gradle.kts') }}-${{ hashFiles('gradle/libs.versions.toml') }}
           restore-keys: ${{ runner.os }}-gradle-caches-
 
-      - name: Setup JDK
+      # Windows does not build ANDROID, but needs Java 11 for JPMS Multi-Release Jar build
+      - name: Setup JDK 11
+        if: matrix.os == 'windows-latest'
         uses: actions/setup-java@v3.4.0
         with:
           distribution: 'zulu'
-          java-version: 17
+          java-version: 11
+
+      - name: Setup JDK 19
+        if: matrix.os != 'windows-latest'
+        uses: actions/setup-java@v3.4.0
+        with:
+          distribution: 'zulu'
+          java-version: 19
 
       - name: Run macOS Tests
         if: matrix.os == 'macos-latest'
@@ -65,7 +74,7 @@ jobs:
           -PKMP_TARGETS="JVM,JS,MINGW_X64,WASM_JS,WASM_WASI"
 
   emulator:
-    runs-on: macos-latest
+    runs-on: ubuntu-latest
 
     strategy:
       fail-fast: false
@@ -75,14 +84,20 @@ jobs:
       - name: Checkout Repo
         uses: actions/checkout@v3
 
+      - name: Enable KVM
+        run: |
+          echo 'KERNEL=="kvm", GROUP="kvm", MODE="0666", OPTIONS+="static_node=kvm"' | sudo tee /etc/udev/rules.d/99-kvm4all.rules
+          sudo udevadm control --reload-rules
+          sudo udevadm trigger --name-match=kvm
+
       - name: Validate Gradle Wrapper
         uses: gradle/wrapper-validation-action@v1
 
       - name: Setup JDK
         uses: actions/setup-java@v3.11.0
         with:
           distribution: 'zulu'
-          java-version: 19
+          java-version: 17
 
       - name: Build
         uses: gradle/gradle-build-action@v2

diff --git a/build-logic/src/main/kotlin/-KmpConfigurationExtension.kt b/build-logic/src/main/kotlin/-KmpConfigurationExtension.kt
@@ -13,24 +13,27 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  **/
+import io.matthewnelson.kmp.configuration.ExperimentalKmpConfigurationApi
 import io.matthewnelson.kmp.configuration.extension.KmpConfigurationExtension
 import io.matthewnelson.kmp.configuration.extension.container.target.KmpConfigurationContainerDsl
 import org.gradle.api.Action
 import org.gradle.api.JavaVersion
 import org.jetbrains.kotlin.gradle.targets.js.dsl.ExperimentalWasmDsl
 
 fun KmpConfigurationExtension.configureShared(
+    java9ModuleName: String? = null,
     publish: Boolean = false,
     explicitApi: Boolean = true,
     action: Action<KmpConfigurationContainerDsl>
 ) {
     configure {
         jvm {
-            target { withJava() }
-
             kotlinJvmTarget = JavaVersion.VERSION_1_8
             compileSourceCompatibility = JavaVersion.VERSION_1_8
             compileTargetCompatibility = JavaVersion.VERSION_1_8
+
+            @OptIn(ExperimentalKmpConfigurationApi::class)
+            java9MultiReleaseModuleInfo(java9ModuleName)
         }
 
         js()

diff --git a/library/md/build.gradle.kts b/library/md/build.gradle.kts
@@ -19,7 +19,7 @@ plugins {
 }
 
 kmpConfiguration {
-    configureShared(publish = true) {
+    configureShared(java9ModuleName = "org.kotlincrypto.hash.md", publish = true) {
         common {
             sourceSetMain {
                 dependencies {

diff --git a/library/md/src/jvmMain/java9/module-info.java b/library/md/src/jvmMain/java9/module-info.java
@@ -0,0 +1,7 @@
+module org.kotlincrypto.hash.md {
+    requires kotlin.stdlib;
+    requires org.kotlincrypto.core;
+    requires org.kotlincrypto.core.digest;
+
+    exports org.kotlincrypto.hash.md;
+}
diff --git a/library/sha1/build.gradle.kts b/library/sha1/build.gradle.kts
@@ -19,7 +19,7 @@ plugins {
 }
 
 kmpConfiguration {
-    configureShared(publish = true) {
+    configureShared(java9ModuleName = "org.kotlincrypto.hash.sha1", publish = true) {
         common {
             sourceSetMain {
                 dependencies {

diff --git a/library/sha1/src/jvmMain/java9/module-info.java b/library/sha1/src/jvmMain/java9/module-info.java
@@ -0,0 +1,8 @@
+@SuppressWarnings("JavaModuleNaming")
+module org.kotlincrypto.hash.sha1 {
+    requires kotlin.stdlib;
+    requires org.kotlincrypto.core;
+    requires org.kotlincrypto.core.digest;
+
+    exports org.kotlincrypto.hash.sha1;
+}
diff --git a/library/sha2/build.gradle.kts b/library/sha2/build.gradle.kts
@@ -19,7 +19,7 @@ plugins {
 }
 
 kmpConfiguration {
-    configureShared(publish = true) {
+    configureShared(java9ModuleName = "org.kotlincrypto.hash.sha2", publish = true) {
         common {
             sourceSetMain {
                 dependencies {

diff --git a/library/sha2/src/jvmMain/java9/module-info.java b/library/sha2/src/jvmMain/java9/module-info.java
@@ -0,0 +1,8 @@
+@SuppressWarnings("JavaModuleNaming")
+module org.kotlincrypto.hash.sha2 {
+    requires kotlin.stdlib;
+    requires org.kotlincrypto.core;
+    requires org.kotlincrypto.core.digest;
+
+    exports org.kotlincrypto.hash.sha2;
+}
diff --git a/library/sha3/build.gradle.kts b/library/sha3/build.gradle.kts
@@ -19,7 +19,7 @@ plugins {
 }
 
 kmpConfiguration {
-    configureShared(publish = true) {
+    configureShared(java9ModuleName = "org.kotlincrypto.hash.sha3", publish = true) {
         common {
             sourceSetMain {
                 dependencies {

diff --git a/library/sha3/src/jvmMain/java9/module-info.java b/library/sha3/src/jvmMain/java9/module-info.java
@@ -0,0 +1,11 @@
+@SuppressWarnings("JavaModuleNaming")
+module org.kotlincrypto.hash.sha3 {
+    requires kotlin.stdlib;
+    requires org.kotlincrypto.core;
+    requires org.kotlincrypto.core.digest;
+    requires org.kotlincrypto.core.xof;
+    requires org.kotlincrypto.endians;
+    requires org.kotlincrypto.sponges.keccak;
+
+    exports org.kotlincrypto.hash.sha3;
+}