Fix more CC compatibility issues

build.gradle

@@ -29,7 +29,7 @@ repositories {
     mavenCentral()
 }
 
-test {
+tasks.withType(Test).configureEach {
     useJUnitPlatform()
 
     if (System.env.CI == 'true') {
@@ -39,6 +39,19 @@ test {
     }
 
     systemProperty 'java.io.tmpdir', buildDir.absolutePath
+
+    // Required to test configuration cache in tests when using withDebug()
+    // https://github.com/gradle/gradle/issues/22765#issuecomment-1339427241
+    jvmArgs(
+            "--add-opens",
+            "java.base/java.util=ALL-UNNAMED",
+            "--add-opens",
+            "java.base/java.util.concurrent.atomic=ALL-UNNAMED",
+            "--add-opens",
+            "java.base/java.lang.invoke=ALL-UNNAMED",
+            "--add-opens",
+            "java.base/java.net=ALL-UNNAMED",
+    )
 }
 
 // jar {

src/main/groovy/com/github/jengelman/gradle/plugins/shadow/ShadowApplicationPlugin.groovy

@@ -93,8 +93,9 @@ class ShadowApplicationPlugin implements Plugin<Project> {
     }
 
     protected void configureInstallTask(Project project) {
-        project.tasks.named(SHADOW_INSTALL_TASK_NAME).configure { installTask ->
-            installTask.doFirst { Sync task ->
+        project.tasks.named(SHADOW_INSTALL_TASK_NAME, Sync).configure { task ->
+            notCompatibleWithConfigurationCache('https://github.com/johnrengelman/shadow/issues/775')
+            task.doFirst {
                 if (task.destinationDir.directory) {
                     if (task.destinationDir.listFiles().size() != 0 && (!new File(task.destinationDir, 'lib').directory || !new File(task.destinationDir, 'bin').directory)) {
                         throw new GradleException("The specified installation directory '${task.destinationDir}' is neither empty nor does it contain an installation for '${javaApplication.applicationName}'.\n" +
@@ -104,7 +105,7 @@ class ShadowApplicationPlugin implements Plugin<Project> {
                     }
                 }
             }
-            installTask.doLast { Sync task ->
+            task.doLast {
                 project.ant.chmod(file: "${task.destinationDir.absolutePath}/bin/${javaApplication.applicationName}", perm: 'ugo+x')
             }
         }

src/main/groovy/com/github/jengelman/gradle/plugins/shadow/transformers/AppendingTransformer.groovy

@@ -41,7 +41,10 @@ class AppendingTransformer implements Transformer {
     @Input
     String resource
 
-    private ByteArrayOutputStream data = new ByteArrayOutputStream()
+    /**
+     * Defer initialization, see https://github.com/johnrengelman/shadow/issues/763
+     */
+    private ByteArrayOutputStream data
 
     boolean canTransformResource(FileTreeElement element) {
         def path = element.relativePath.pathString
@@ -53,17 +56,25 @@ class AppendingTransformer implements Transformer {
     }
 
     void transform(TransformerContext context) {
+        if (data == null) {
+            data = new ByteArrayOutputStream()
+        }
+
         IOUtil.copy(context.is, data)
         data.write('\n'.bytes)
 
         context.is.close()
     }
 
     boolean hasTransformedResource() {
-        return data.size() > 0
+        return data?.size() > 0
     }
 
     void modifyOutputStream(ZipOutputStream os, boolean preserveFileTimestamps) {
+        if (data == null) {
+            data = new ByteArrayOutputStream()
+        }
+
         ZipEntry entry = new ZipEntry(resource)
         entry.time = TransformerContext.getEntryTimestamp(preserveFileTimestamps, entry.time)
         os.putNextEntry(entry)

src/test/groovy/com/github/jengelman/gradle/plugins/shadow/ApplicationSpec.groovy

@@ -43,7 +43,7 @@ class ApplicationSpec extends PluginSpecification {
         settingsFile << "rootProject.name = 'myapp'"
 
         when:
-        BuildResult result = run('runShadow', '--stacktrace')
+        BuildResult result = run('runShadow')
 
         then: 'tests that runShadow executed and exited'
         assert result.output.contains('TestApp: Hello World! (foo)')
@@ -105,7 +105,7 @@ class ApplicationSpec extends PluginSpecification {
             runShadow {
                args 'foo'
                doFirst {
-                project.logger.lifecycle("Running application with JDK \${it.javaLauncher.get().metadata.languageVersion.asInt()}")
+                   logger.lifecycle("Running application with JDK \${it.javaLauncher.get().metadata.languageVersion.asInt()}")
                }
             }          
         """.stripIndent()
@@ -120,7 +120,7 @@ class ApplicationSpec extends PluginSpecification {
         """.stripIndent()
 
         when:
-        BuildResult result = run('runShadow', '--stacktrace')
+        BuildResult result = run('runShadow')
 
         then: 'tests that runShadow executed and exited'
         assert result.output.contains('Running application with JDK 17')
@@ -183,7 +183,7 @@ class ApplicationSpec extends PluginSpecification {
         settingsFile << "rootProject.name = 'myapp'"
 
         when:
-        run('shadowDistZip', '--stacktrace')
+        run('shadowDistZip')
 
         then: 'Check that the distribution zip was created'
         File zip = getFile('build/distributions/myapp-shadow-1.0.zip')

src/test/groovy/com/github/jengelman/gradle/plugins/shadow/ConfigurationCacheSpec.groovy

@@ -49,8 +49,8 @@ class ConfigurationCacheSpec extends PluginSpecification {
         settingsFile << "rootProject.name = 'myapp'"
 
         when:
-        run('--configuration-cache', 'shadowJar')
-        def result = run('--configuration-cache', 'shadowJar')
+        run('shadowJar')
+        def result = run('shadowJar')
 
         then:
         result.output.contains("Reusing configuration cache.")
@@ -65,9 +65,9 @@ class ConfigurationCacheSpec extends PluginSpecification {
         """.stripIndent()
 
         when:
-        run('--configuration-cache', 'shadowJar')
+        run('shadowJar')
         output.delete()
-        def result = run('--configuration-cache', 'shadowJar')
+        def result = run('shadowJar')
 
         then:
         contains(output, ['a.properties', 'b.properties'])
@@ -117,9 +117,9 @@ class ConfigurationCacheSpec extends PluginSpecification {
         def output = getFile('server/build/libs/server-all.jar')
 
         when:
-        run('--configuration-cache', 'shadowJar', '-s')
+        run('shadowJar', '-s')
         output.delete()
-        def result = run('--configuration-cache', 'shadowJar', '-s')
+        def result = run('shadowJar', '-s')
 
         then:
         output.exists()
@@ -160,8 +160,8 @@ class ConfigurationCacheSpec extends PluginSpecification {
         """.stripIndent()
 
         when:
-        run('--configuration-cache', 'shadowJar', '-s')
-        def result = run('--configuration-cache', 'shadowJar', '-s')
+        run('shadowJar', '-s')
+        def result = run('shadowJar', '-s')
 
         then:
         result.output.contains(":lib:shadowJar UP-TO-DATE")

src/test/groovy/com/github/jengelman/gradle/plugins/shadow/PublishingSpec.groovy

@@ -52,6 +52,10 @@ class PublishingSpec extends PluginSpecification {
                    }
                }
             }
+
+            tasks.withType(org.gradle.api.publish.maven.tasks.GenerateMavenPom).configureEach {
+                notCompatibleWithConfigurationCache('https://github.com/gradle/gradle/issues/21926')
+            }
         """.stripIndent()
 
         when:
@@ -149,6 +153,10 @@ class PublishingSpec extends PluginSpecification {
                    }
                }
             }
+
+            tasks.withType(org.gradle.api.publish.maven.tasks.GenerateMavenPom).configureEach {
+                notCompatibleWithConfigurationCache('https://github.com/gradle/gradle/issues/21926')
+            }
         """.stripMargin()
 
         when:

src/test/groovy/com/github/jengelman/gradle/plugins/shadow/ShadowPluginSpec.groovy

@@ -256,7 +256,7 @@ class ShadowPluginSpec extends PluginSpecification {
         File serverOutput = getFile('server/build/libs/server-all.jar')
 
         when:
-        runWithDebug(':server:shadowJar', '--stacktrace')
+        runWithDebug(':server:shadowJar')
 
         then:
         serverOutput.exists()
@@ -311,7 +311,7 @@ class ShadowPluginSpec extends PluginSpecification {
         File serverOutput = getFile('server/build/libs/server-all.jar')
 
         when:
-        runWithDebug(':server:shadowJar', '--stacktrace')
+        runWithDebug(':server:shadowJar')
 
         then:
         serverOutput.exists()
@@ -364,7 +364,7 @@ class ShadowPluginSpec extends PluginSpecification {
         File serverOutput = file('server/build/libs/server-all.jar')
 
         when:
-        runWithDebug(':server:shadowJar', '--stacktrace')
+        runWithDebug(':server:shadowJar')
 
         then:
         contains(serverOutput, [
@@ -419,7 +419,7 @@ class ShadowPluginSpec extends PluginSpecification {
         File serverOutput = file('server/build/libs/server-all.jar')
 
         when:
-        runWithDebug(':server:shadowJar', '--stacktrace')
+        runWithDebug(':server:shadowJar')
 
         then:
         contains(serverOutput, [
@@ -472,7 +472,7 @@ class ShadowPluginSpec extends PluginSpecification {
         File serverOutput = file('server/build/libs/server-all.jar')
 
         when:
-        runWithDebug(':server:shadowJar', '--stacktrace')
+        runWithDebug(':server:shadowJar')
 
         then:
         contains(serverOutput, [
@@ -550,7 +550,7 @@ class ShadowPluginSpec extends PluginSpecification {
         File serverOutput = getFile('impl/build/libs/impl-all.jar')
 
         when:
-        runWithDebug(':impl:shadowJar', '--stacktrace')
+        runWithDebug(':impl:shadowJar')
 
         then:
         serverOutput.exists()
@@ -626,7 +626,7 @@ class ShadowPluginSpec extends PluginSpecification {
         File serverOutput = getFile('impl/build/libs/impl-all.jar')
 
         when:
-        runWithDebug(':impl:shadowJar', '--stacktrace')
+        runWithDebug(':impl:shadowJar')
 
         then:
         serverOutput.exists()
@@ -992,7 +992,7 @@ class ShadowPluginSpec extends PluginSpecification {
         """.stripIndent()
 
         when:
-        run('shadowJar', '--stacktrace')
+        run('shadowJar')
 
         then:
         assert output.exists()
@@ -1040,7 +1040,7 @@ class ShadowPluginSpec extends PluginSpecification {
         File serverOutput = getFile('impl/build/libs/impl-1.0-all.jar')
 
         when:
-        runWithDebug(':impl:shadowJar', '--stacktrace')
+        runWithDebug(':impl:shadowJar')
 
         then:
         serverOutput.exists()
@@ -1112,7 +1112,7 @@ class ShadowPluginSpec extends PluginSpecification {
         settingsFile << "rootProject.name = 'myapp'"
 
         when:
-        BuildResult result = run('runShadow', '--stacktrace')
+        BuildResult result = run('runShadow')
 
         then: 'tests that runShadow executed and exited'
         assert result.output.contains('TestApp: Hello World! (foo)')
@@ -1170,7 +1170,7 @@ class ShadowPluginSpec extends PluginSpecification {
         """.stripIndent()
 
         when:
-        BuildResult result = run('runShadow', '--stacktrace')
+        BuildResult result = run('runShadow')
 
         then: 'tests that runShadow executed and exited'
         assert result.output.contains('TestApp: Hello World! (foo)')
@@ -1212,7 +1212,7 @@ class ShadowPluginSpec extends PluginSpecification {
         """.stripIndent()
 
         when:
-        BuildResult result = run('runShadow', '--stacktrace')
+        BuildResult result = run('runShadow')
 
         then: 'tests that runShadow executed and exited'
         assert result.output.contains('TestApp: Hello World! (foo)')

src/test/groovy/com/github/jengelman/gradle/plugins/shadow/TransformerSpec.groovy

@@ -745,7 +745,7 @@ staticExtensionClasses=com.acme.bar.SomeStaticExtension'''.stripIndent()).write(
         """.stripIndent()
 
         when:
-        run('shadowJar', '--warning-mode=all', '--stacktrace')
+        run('shadowJar', '--warning-mode=all')
 
         then:
         assert output.exists()

src/test/groovy/com/github/jengelman/gradle/plugins/shadow/caching/TransformCachingSpec.groovy

@@ -47,6 +47,7 @@ class TransformCachingSpec extends AbstractCachingSpec {
             }
 
             shadowJar {
+                notCompatibleWithConfigurationCache('CustomTransformer is not cacheable')
                 transform(CustomTransformer)
             }
         """.stripIndent()

src/test/groovy/com/github/jengelman/gradle/plugins/shadow/util/PluginSpecification.groovy

@@ -82,7 +82,7 @@ class PluginSpecification extends Specification {
     }
 
     GradleRunner runner(Collection<String> tasks) {
-        runner.withArguments(["-Dorg.gradle.warning.mode=all"] + tasks.toList())
+        runner.withArguments(["-Dorg.gradle.warning.mode=all", "--configuration-cache", "--stacktrace"] + tasks.toList())
     }
 
     void assertNoDeprecationWarnings(BuildResult result) {