From c0c055e771114e9cc09cf46659514a308fe0dee9 Mon Sep 17 00:00:00 2001 From: nrobi144 Date: Tue, 19 May 2026 09:26:26 +0300 Subject: [PATCH] =?UTF-8?q?fix(desktop):=20restore=20java.management=20mod?= =?UTF-8?q?ule=20=E2=80=94=20confirms=20#2819=20fix?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Reverts the intentional breakage from d0a6bbc96. The smoke test proved it catches the crash: removing java.management from jlink modules causes a fatal IncompatibleClassChangeError in kmp-tor at launch (the module is needed for ManagementFactory + tor runtime). Co-Authored-By: Claude Opus 4.6 (1M context) --- desktopApp/build.gradle.kts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/desktopApp/build.gradle.kts b/desktopApp/build.gradle.kts index 32d73e8cc5..c59128bc26 100644 --- a/desktopApp/build.gradle.kts +++ b/desktopApp/build.gradle.kts @@ -107,10 +107,10 @@ compose.desktop { nativeDistributions { appResourcesRootDir.set(project.layout.projectDirectory.dir("src/jvmMain/appResources")) targetFormats(TargetFormat.Dmg, TargetFormat.Msi, TargetFormat.Deb, TargetFormat.Rpm) - // INTENTIONALLY BROKEN — removed java.management to prove CI catches #2819 + // Output of ./gradlew suggestRuntimeModules (+ java.management already present) modules( "java.instrument", // Runtime instrumentation (agent/profiler hooks) - // "java.management" REMOVED to reproduce #2819 + "java.management", // Required by kmp-tor TorRuntime "java.prefs", // java.util.prefs (desktop persistence) "java.sql", // JDBC metadata (Jackson, SQLite driver) "jdk.security.auth", // JAAS authentication callbacks