commit b400b027c90675f19b03c08d937b51e518220ca5
author Vsevolod Tolstopyatov <qwwdfsad@gmail.com> Wed Apr 21 17:42:23 2021 +0300
committer GitHub <noreply@github.com> Wed Apr 21 17:42:23 2021 +0300
tree 632eac9a7221e5aec6739e503e6d1b2c0504e44d
parent e49cf79679ef742c08588ec572f55289f4474f46

Add debugger-related manifest to root metadata artifact of kotlinx-coroutines-core (#2632)

* It fixes IDEA issue that was introduced in coroutines 1.4.3 where we switched to Kotlin 1.4.30 and stopped producing root artifact with JVM classes
    * It simplifies manual -javaagent specification for end-users who shouldn't even be aware of kotlinx-coroutines-core-jvm.jar

Fixes #2619

kotlinx-coroutines-core/build.gradle

diff --git a/kotlinx-coroutines-core/build.gradle b/kotlinx-coroutines-core/build.gradle
index c2a57f9..4ea81d3 100644
--- a/kotlinx-coroutines-core/build.gradle
+++ b/kotlinx-coroutines-core/build.gradle
@@ -208,8 +208,20 @@
     }
 }
 
-jvmJar {
-    manifest {
+// Setup manifest for kotlinx-coroutines-core-jvm.jar
+jvmJar { setupManifest(it) }
+
+/*
+ * Setup manifest for kotlinx-coroutines-core.jar
+ * This is convenient for users that pass -javaagent arg manually and also is a workaround #2619 and KTIJ-5659.
+ * This manifest contains reference to AgentPremain that belongs to
+ * kotlinx-coroutines-core-jvm, but our resolving machinery guarantees that
+ * any JVM project that depends on -core artifact also depends on -core-jvm one.
+ */
+metadataJar { setupManifest(it) }
+
+static def setupManifest(Jar jar) {
+    jar.manifest {
         attributes "Premain-Class": "kotlinx.coroutines.debug.AgentPremain"
         attributes "Can-Retransform-Classes": "true"
     }