Capture throwable inside generateDocumentation thread (#2935)

Fixes #2934
Kotlin · Apr 17, 2023 · 115a318 · 115a318
1 parent 5e81202
commit 115a318
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/runners/gradle-plugin/src/main/kotlin/org/jetbrains/dokka/gradle/tasks/AbstractDokkaTask.kt b/runners/gradle-plugin/src/main/kotlin/org/jetbrains/dokka/gradle/tasks/AbstractDokkaTask.kt
@@ -20,6 +20,7 @@ import org.gradle.work.DisableCachingByDefault
 import org.jetbrains.dokka.*
 import org.jetbrains.dokka.plugability.ConfigurableBlock
 import org.jetbrains.dokka.plugability.DokkaPlugin
+import java.util.concurrent.atomic.AtomicReference
 import java.util.function.BiConsumer
 import kotlin.reflect.full.createInstance
 
@@ -203,14 +204,17 @@ abstract class AbstractDokkaTask : DefaultTask() {
     internal open fun generateDocumentation() {
         DokkaBootstrap(runtime, DokkaBootstrapImpl::class).apply {
             configure(buildDokkaConfiguration().toCompactJsonString(), createProxyLogger())
+            val uncaughtExceptionHolder = AtomicReference<Throwable?>()
             /**
              * Run in a new thread to avoid memory leaks that are related to ThreadLocal (that keeps `URLCLassLoader`)
              * Currently, all `ThreadLocal`s leaking are in the compiler/IDE codebase.
              */
             Thread { generate() }.apply {
+                setUncaughtExceptionHandler { _, throwable -> uncaughtExceptionHolder.set(throwable) }
                 start()
                 join()
             }
+            uncaughtExceptionHolder.get()?.let { throw it }
         }
     }