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/docs/publishing/README.md

@@ -16,7 +16,7 @@ apply plugin: 'com.github.johnrengelman.shadow'
 publishing {
   publications {
     shadow(MavenPublication) { publication ->
-      project.shadow.component(publication)
+      project.shadow.component(project, publication)
     }
   }
   repositories {

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

@@ -93,8 +93,8 @@ 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 ->
+            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,8 +104,12 @@ class ShadowApplicationPlugin implements Plugin<Project> {
                     }
                 }
             }
-            installTask.doLast { Sync task ->
-                project.ant.chmod(file: "${task.destinationDir.absolutePath}/bin/${javaApplication.applicationName}", perm: 'ugo+x')
+            task.doLast {
+                task.eachFile {
+                    if (it.path == "bin/${javaApplication.applicationName}") {
+                        it.mode = 0x755
+                    }
+                }
             }
         }
     }

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

@@ -16,7 +16,7 @@ class ShadowBasePlugin implements Plugin<Project> {
         if (GradleVersion.current() < GradleVersion.version("8.0")) {
             throw new GradleException("This version of Shadow supports Gradle 8.0+ only. Please upgrade.")
         }
-        project.extensions.create(EXTENSION_NAME, ShadowExtension, project)
+        project.extensions.create(EXTENSION_NAME, ShadowExtension)
         createShadowConfiguration(project)
 
         project.tasks.register(KnowsTask.NAME, KnowsTask) { knows ->

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

@@ -3,42 +3,50 @@ package com.github.jengelman.gradle.plugins.shadow
 import org.gradle.api.Project
 import org.gradle.api.artifacts.ProjectDependency
 import org.gradle.api.artifacts.SelfResolvingDependency
-import org.gradle.api.file.CopySpec
 import org.gradle.api.publish.maven.MavenPom
 import org.gradle.api.publish.maven.MavenPublication
 import org.gradle.util.GradleVersion
 
 class ShadowExtension {
 
-    CopySpec applicationDistribution
-    Project project
-
-    ShadowExtension(Project project) {
-        this.project = project
-        applicationDistribution = project.copySpec {}
-    }
-
-    void component(MavenPublication publication) {
+    void component(Project project, MavenPublication publication) {
 
         if (GradleVersion.current() >= GradleVersion.version("6.6")) {
             publication.artifact(project.tasks.named("shadowJar"))
         } else {
             publication.artifact(project.tasks.shadowJar)
         }
 
+        final def allDependencies = project.provider {
+            project.configurations.shadow.allDependencies.collect {
+                if ((it instanceof ProjectDependency) || ! (it instanceof SelfResolvingDependency)) {
+                    new Dep(it.group, it.name, it.version)
+                }
+            }
+        }
         publication.pom { MavenPom pom ->
             pom.withXml { xml ->
                 def dependenciesNode = xml.asNode().get('dependencies') ?: xml.asNode().appendNode('dependencies')
-                project.configurations.shadow.allDependencies.each {
-                    if ((it instanceof ProjectDependency) || ! (it instanceof SelfResolvingDependency)) {
-                        def dependencyNode = dependenciesNode.appendNode('dependency')
-                        dependencyNode.appendNode('groupId', it.group)
-                        dependencyNode.appendNode('artifactId', it.name)
-                        dependencyNode.appendNode('version', it.version)
-                        dependencyNode.appendNode('scope', 'runtime')
-                    }
+                allDependencies.get().each {
+                    def dependencyNode = dependenciesNode.appendNode('dependency')
+                    dependencyNode.appendNode('groupId', it.group)
+                    dependencyNode.appendNode('artifactId', it.name)
+                    dependencyNode.appendNode('version', it.version)
+                    dependencyNode.appendNode('scope', 'runtime')
                 }
             }
         }
     }
+
+    private class Dep {
+        String group
+        String name
+        String version
+
+        Dep(String group, String name, String version) {
+            this.group = group
+            this.name = name
+            this.version = version
+        }
+    }
 }

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

@@ -42,7 +42,7 @@ class PublishingSpec extends PluginSpecification {
             publishing {
                publications {
                    shadow(MavenPublication) { publication ->
-                       project.shadow.component(publication)
+                       project.shadow.component(project, publication)
                        artifactId = 'maven-all'
                    }
                }
@@ -144,7 +144,7 @@ class PublishingSpec extends PluginSpecification {
             publishing {
                publications {
                    shadow(MavenPublication) { publication ->
-                       project.shadow.component(publication)
+                       project.shadow.component(project, publication)
                        artifactId = 'maven-all'
                    }
                }

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()