From f665b6eed4299da5adf4978a8b872d305699a934 Mon Sep 17 00:00:00 2001 From: Denis Vaumoron Date: Sun, 22 Dec 2024 23:11:52 +0100 Subject: [PATCH] Change default for TENV_DETACHED_PROXY on non windows OS #255 (#304) Signed-off-by: Denis Vaumoron --- README.md | 4 ++-- versionmanager/proxy/detach/detached_others.go | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index e86d513..60aa90a 100644 --- a/README.md +++ b/README.md @@ -639,9 +639,9 @@ If set to true **tenv** will automatically install missing tool versions needed.
TENV_DETACHED_PROXY
-String (Default: false) +String (Default: false on Windows OS, true on other OS) -Enable **tenv** proxies detached behaviour (use a new process group id when launching tool command, allow to avoid killing proxied tool process earlier than expected when proxies redirect interrupt signal already sended to whole process group (issue occurrences known on macOS)). +Enable **tenv** proxies detached behaviour (use a new process group id when launching tool command, allow to avoid killing proxied tool process earlier than expected when proxies redirect interrupt signal already sended to whole process group (issue occurrences known on macOS and Linux)).
diff --git a/versionmanager/proxy/detach/detached_others.go b/versionmanager/proxy/detach/detached_others.go index 2eea42f..d9035ec 100644 --- a/versionmanager/proxy/detach/detached_others.go +++ b/versionmanager/proxy/detach/detached_others.go @@ -32,7 +32,7 @@ import ( const msgErr = "Failed to read " + config.TenvDetachedProxyEnvName + " environment variable, disable behavior :" func InitBehaviorFromEnv(cmd *exec.Cmd, getenv configutils.GetenvFunc) { - detached, err := getenv.Bool(false, config.TenvDetachedProxyEnvName) + detached, err := getenv.Bool(true, config.TenvDetachedProxyEnvName) if err != nil { fmt.Println(msgErr, err) //nolint }