Merge pull request #352 from GordonSmith/GH-351

fix: Clienttools terminal will fail on *nix platforms
hpcc-systems · Aug 24, 2023 · 6fa41a2 · 6fa41a2
2 parents 51e511e + f1e0a7a
commit 6fa41a2
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/ecl/terminal.ts b/src/ecl/terminal.ts
@@ -1,13 +1,16 @@
 import { ClientTools } from "@hpcc-js/comms";
 import * as vscode from "vscode";
+import * as os from "os";
 import { sessionManager } from "../hpccplatform/session";
 
+const PATH_SEP = os.platform() === "win32" ? ";" : ":";
+
 export function eclTerminal(ct: ClientTools) {
     const ver = ct.versionSync();
     const terminal = vscode.window.createTerminal({
         name: `ECL v${ver.major}.${ver.minor}.${ver.patch}`,
         env: {
-            PATH: `${ct.binPath};${process.env.PATH}`
+            PATH: `${ct.binPath}${PATH_SEP}${process.env.PATH}`
         }
     });
     terminal.show();