fix: cross-platform process signal for Windows goreleaser build

garagon · garagon · commit ff751cd2ba49 · 2026-03-18T11:25:40.000-03:00
diff --git a/cmd/oktsec/commands/run.go b/cmd/oktsec/commands/run.go
@@ -461,9 +461,11 @@ func killExistingInstance() {
 		}
 		var other int
 		if _, err := fmt.Sscanf(line, "%d", &other); err == nil && other != pid {
-			_ = syscall.Kill(other, syscall.SIGTERM)
-			// Brief wait for graceful shutdown.
-			time.Sleep(200 * time.Millisecond)
+			if p, err := os.FindProcess(other); err == nil {
+				_ = p.Signal(os.Interrupt)
+				// Brief wait for graceful shutdown.
+				time.Sleep(200 * time.Millisecond)
+			}
 		}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -461,9 +461,11 @@ func killExistingInstance() {`
`461`	`461`	`}`
`462`	`462`	`var other int`
`463`	`463`	`if _, err := fmt.Sscanf(line, "%d", &other); err == nil && other != pid {`
`464`		`- _ = syscall.Kill(other, syscall.SIGTERM)`
`465`		`- // Brief wait for graceful shutdown.`
`466`		`- time.Sleep(200 * time.Millisecond)`
	`464`	`+ if p, err := os.FindProcess(other); err == nil {`
	`465`	`+ _ = p.Signal(os.Interrupt)`
	`466`	`+ // Brief wait for graceful shutdown.`
	`467`	`+ time.Sleep(200 * time.Millisecond)`
	`468`	`+ }`
`467`	`469`	`}`
`468`	`470`	`}`
`469`	`471`	`}`