Skip to content

Commit

Permalink
Revert "build: update JPMS patch rules (#42)"
Browse files Browse the repository at this point in the history
This reverts commit 8550d2a.

Signed-off-by: Jendrik Johannes <[email protected]>
  • Loading branch information
jjohannes committed Dec 20, 2024
1 parent 8550d2a commit a73bec3
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 69 deletions.
8 changes: 0 additions & 8 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,13 +1,5 @@
# Hiero Gradle Conventions - Changelog

## Version 0.2.0

* JPMS - remove patching rule: commons-codec:commons-codec
* JPMS - remove patching rule: org.jetbrains:annotations
* JPMS - remove patching rule: org.json:json
* JPMS - remove patching rule: errorprone (rules were not creating valid modules)
* JPMS - adjust module name: com.google.common.jimfs

## Version 0.1.2

* make license header configurable via 'license-header.txt' (#48)
Expand Down
23 changes: 22 additions & 1 deletion src/main/kotlin/org.hiero.gradle.base.jpms-modules.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,7 @@ extraJavaModuleInfo {
module("com.google.dagger:dagger", "dagger")
module("io.perfmark:perfmark-api", "io.perfmark")
module("javax.inject:javax.inject", "javax.inject")
module("commons-codec:commons-codec", "org.apache.commons.codec")
module("com.esaulpaugh:headlong", "com.esaulpaugh.headlong") {
exportAllPackages()
overrideModuleName() // for older versions with 'Automatic-Module-Name: headlong'
Expand All @@ -136,6 +137,7 @@ extraJavaModuleInfo {
exportAllPackages()
requires("org.connid.framework") // this is missing in POM
}
module("org.jetbrains:annotations", "org.jetbrains.annotations")
module("io.tmio:tuweni-units", "tuweni.units")
module("io.tmio:tuweni-bytes", "tuweni.bytes")
module("net.i2p.crypto:eddsa", "net.i2p.crypto.eddsa")
Expand Down Expand Up @@ -215,6 +217,24 @@ extraJavaModuleInfo {
module("net.ltgt.gradle.incap:incap", "net.ltgt.gradle.incap")
module("org.jetbrains.kotlinx:kotlinx-metadata-jvm", "kotlinx.metadata.jvm")

// Annotation processing - error prone
module(
"com.github.kevinstern:software-and-algorithms",
"com.github.kevinstern.software_and_algorithms"
)
module("com.google.auto.value:auto-value-annotations", "com.google.auto.value.annotations")
module("com.google.errorprone:error_prone_annotation", "com.google.errorprone.annotation")
module("com.google.errorprone:error_prone_check_api", "com.google.errorprone.check.api")
module("com.google.errorprone:error_prone_core", "com.google.errorprone.core")
module(
"com.google.errorprone:error_prone_type_annotations",
"com.google.errorprone.type.annotations"
)
module("com.uber.nullaway:nullaway", "com.uber.nullaway")
module("io.github.eisop:dataflow-errorprone", "org.checkerframework.dataflow")
module("io.github.java-diff-utils:java-diff-utils", "io.github.javadiffutils")
module("io.grpc:grpc-java-api-checker", "io.grpc.java.api.checker")

// Testing only
module("io.grpc:grpc-netty-shaded", "io.grpc.netty.shaded") {
exportAllPackages()
Expand All @@ -223,7 +243,7 @@ extraJavaModuleInfo {
requires("jdk.unsupported")
ignoreServiceProvider("reactor.blockhound.integration.BlockHoundIntegration")
}
module("com.google.jimfs:jimfs", "com.google.common.jimfs")
module("com.google.jimfs:jimfs", "com.google.jimfs")
module("io.github.json-snapshot:json-snapshot", "json.snapshot")
module("org.awaitility:awaitility", "awaitility")
module("uk.org.webcompere:system-stubs-core", "uk.org.webcompere.systemstubs.core")
Expand All @@ -243,6 +263,7 @@ extraJavaModuleInfo {
}
module("junit:junit", "junit")
module("org.hamcrest:hamcrest", "org.hamcrest")
module("org.json:json", "org.json")
module("org.mockito:mockito-core", "org.mockito")
module("org.objenesis:objenesis", "org.objenesis")
module("org.rnorth.duct-tape:duct-tape", "org.rnorth.ducttape")
Expand Down
60 changes: 0 additions & 60 deletions src/test/kotlin/org/hiero/gradle/test/JpmsPatchTest.kt

This file was deleted.

0 comments on commit a73bec3

Please sign in to comment.