apache#7606: Supply empty stdin, process daemon's stdout to avoid lockup

sdedic · Sep 2, 2024 · 71d7936 · 71d7936
1 parent f59beaf
commit 71d7936
Showing 1 changed file with 18 additions and 14 deletions.
diff --git a/extide/gradle/src/org/netbeans/modules/gradle/loaders/LegacyProjectLoader.java b/extide/gradle/src/org/netbeans/modules/gradle/loaders/LegacyProjectLoader.java
@@ -19,6 +19,7 @@
 package org.netbeans.modules.gradle.loaders;
 
 import java.io.BufferedReader;
+import java.io.ByteArrayInputStream;
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
@@ -636,21 +637,24 @@ public void run() {
 
         OutputStream logStream = null;
         try {
-            if (LOG.isLoggable(Level.FINER)) {
-                if (LOG.isLoggable(Level.FINEST)) {
-                    action.addArguments("--debug"); // NOI18N
-                }
-                PipedOutputStream pos = new ImmediatePipedOutputStream();
-                try {
-                    logStream = pos;
-                    DAEMON_LOG_RP.post(new LogDelegate(new PipedInputStream(pos)));
-                } catch (IOException ex) {
-                    throw new IllegalStateException(ex);
-                }
-                action.setStandardOutput(pos);
-                action.setStandardError(pos);
+            // no input will be ever given ...
+            InputStream emptyIs = new ByteArrayInputStream(new byte[0]);
+            // Github #7606: although the content of the OuptutStream will not be printed unless LOG is >= FINER,
+            // for some reason gradle daemon blocks on (empty) stdin if the output is not read+processed.
+            // So attaching the daemon + output stream handler unconditionally.
+            if (LOG.isLoggable(Level.FINEST)) {
+                action.addArguments("--debug"); // NOI18N
             }
-
+            PipedOutputStream pos = new ImmediatePipedOutputStream();
+            try {
+                logStream = pos;
+                DAEMON_LOG_RP.post(new LogDelegate(new PipedInputStream(pos)));
+            } catch (IOException ex) {
+                throw new IllegalStateException(ex);
+            }
+            action.setStandardOutput(pos);
+            action.setStandardError(pos);
+            action.setStandardInput(emptyIs);
             return action.run(); 
         } finally {
             if (logStream != null) {