Start/remove dbg string (#46)

bcumming · web-flow · commit 4fb8523df156 · 2025-01-10T12:47:56.000+01:00
* remove redundant debug string and remove eroneous PS1 detection
* VERSION = 7.0.0
diff --git a/VERSION b/VERSION
@@ -1 +1 @@
-7.0.0-dev
+7.0.0
diff --git a/src/cli/start.cpp b/src/cli/start.cpp
@@ -52,9 +52,7 @@ std::optional<std::string> detect_non_interactive() {
     if (std::getenv("BASH_EXECUTION_STRING")) {
         return "BASH_EXECUTION_STRING is set";
     }
-    if (!std::getenv("PS1")) {
-        return "PS1 is not set";
-    }
+    // don't check PS1 - PS1 is not exported by tools like oh-my-posh in zsh
     return std::nullopt;
 }
 
@@ -82,8 +80,6 @@ If your intention is to initialize an environment (like module load), uenv start
 will not work, because it starts a new interactive shell.)",
                       reason.value(), "uenv run --help"));
         return 1;
-    } else {
-        fmt::println("uenv start worked!");
     }
 
     const auto env = concretise_env(args.uenv_description,

Original file line number	Diff line number	Diff line change
`@@ -52,9 +52,7 @@ std::optional<std::string> detect_non_interactive() {`
`52`	`52`	`if (std::getenv("BASH_EXECUTION_STRING")) {`
`53`	`53`	`return "BASH_EXECUTION_STRING is set";`
`54`	`54`	`}`
`55`		`- if (!std::getenv("PS1")) {`
`56`		`- return "PS1 is not set";`
`57`		`- }`
	`55`	`+ // don't check PS1 - PS1 is not exported by tools like oh-my-posh in zsh`
`58`	`56`	`return std::nullopt;`
`59`	`57`	`}`
`60`	`58`
`@@ -82,8 +80,6 @@ If your intention is to initialize an environment (like module load), uenv start`
`82`	`80`	`will not work, because it starts a new interactive shell.)",`
`83`	`81`	`reason.value(), "uenv run --help"));`
`84`	`82`	`return 1;`
`85`		`- } else {`
`86`		`- fmt::println("uenv start worked!");`
`87`	`83`	`}`
`88`	`84`
`89`	`85`	`const auto env = concretise_env(args.uenv_description,`