Fix merge

GradleUp · Nov 4, 2024 · d5863b9 · d5863b9
1 parent 180e173
commit d5863b9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/build-logic/src/main/kotlin/shadow.convention.publish.gradle.kts b/build-logic/src/main/kotlin/shadow.convention.publish.gradle.kts
@@ -45,8 +45,6 @@ configurations {
   listOf(
     apiElements,
     runtimeElements,
-    named("javadocElements"),
-    named("sourcesElements"),
   ).forEach {
     it.configure {
       outgoing {

diff --git a/src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/ShadowPlugin.kt b/src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/ShadowPlugin.kt
@@ -11,13 +11,18 @@ public class ShadowPlugin : Plugin<Project> {
   override fun apply(project: Project) {
     with(project.plugins) {
       apply(ShadowBasePlugin::class.java)
-      apply(LegacyShadowPlugin::class.java)
       withType(JavaPlugin::class.java) {
         apply(ShadowJavaPlugin::class.java)
       }
       withType(ApplicationPlugin::class.java) {
         apply(ShadowApplicationPlugin::class.java)
       }
+      // Apply the legacy plugin last
+      //   Because we apply the ShadowJavaPlugin/ShadowApplication plugin in a withType callback for the
+      //   respective JavaPlugin/ApplicationPlugin, it may still apply before the shadowJar task is created and
+      //   etc. if the user applies shadow before those plugins. However, this is fine, because this was also
+      //   the behavior with the old plugin when applying in that order.
+      apply(LegacyShadowPlugin::class.java)
     }
   }
 }