Fixed task hang up in shellInternal due to unread error pipe.

spotify · Nov 20, 2023 · 4330b9b · 4330b9b
1 parent 64472d5
commit 4330b9b
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/Sources/XCRemoteCache/Shell/Shell.swift b/Sources/XCRemoteCache/Shell/Shell.swift
@@ -84,13 +84,13 @@ private func shellInternal(_ cmd: String, args: [String] = [], stdout: PipeLike?
         task.currentDirectoryPath = dir
     }
     task.launch()
+    let errorData = errorHandle.fileHandleForReading.readDataToEndOfFile()
     task.waitUntilExit()
     if task.terminationStatus != 0 {
         if stderr != nil {
             // Error stream was captured so cannot inspect its content
             throw ShellError.statusError("Failed command", task.terminationStatus)
         }
-        let errorData = errorHandle.fileHandleForReading.readDataToEndOfFile()
         let errorString = String(data: errorData, encoding: .utf8)?.trim() ?? "No error returned from the process."
         throw ShellError.statusError(
             "status \(task.terminationStatus): \(errorString)", task.terminationStatus