Fixed bug where child process did not set parent process back to foreground

John Bute · John Bute · commit 9826fc2cc4c5 · 2025-02-11T11:21:55.000-05:00
diff --git a/Sources/SwiftlyCore/Platform.swift b/Sources/SwiftlyCore/Platform.swift
@@ -211,6 +211,10 @@ extension Platform {
         }
         process.waitUntilExit()
 
+        if pgid != -1 {
+            tcsetpgrp(STDOUT_FILENO, pgid)
+        }
+
         guard process.terminationStatus == 0 else {
             throw RunProgramError(exitCode: process.terminationStatus, program: args.first!)
         }
@@ -254,6 +258,10 @@ extension Platform {
 
         process.waitUntilExit()
 
+        if pgid != -1 {
+            tcsetpgrp(STDOUT_FILENO, pgid)
+        }
+
         guard process.terminationStatus == 0 else {
             throw RunProgramError(exitCode: process.terminationStatus, program: args.first!)
         }

Original file line number	Diff line number	Diff line change
`@@ -211,6 +211,10 @@ extension Platform {`
`211`	`211`	`}`
`212`	`212`	`process.waitUntilExit()`
`213`	`213`
	`214`	`+ if pgid != -1 {`
	`215`	`+ tcsetpgrp(STDOUT_FILENO, pgid)`
	`216`	`+ }`
	`217`	`+`
`214`	`218`	`guard process.terminationStatus == 0 else {`
`215`	`219`	`throw RunProgramError(exitCode: process.terminationStatus, program: args.first!)`
`216`	`220`	`}`
`@@ -254,6 +258,10 @@ extension Platform {`
`254`	`258`
`255`	`259`	`process.waitUntilExit()`
`256`	`260`
	`261`	`+ if pgid != -1 {`
	`262`	`+ tcsetpgrp(STDOUT_FILENO, pgid)`
	`263`	`+ }`
	`264`	`+`
`257`	`265`	`guard process.terminationStatus == 0 else {`
`258`	`266`	`throw RunProgramError(exitCode: process.terminationStatus, program: args.first!)`
`259`	`267`	`}`