aboutsummaryrefslogtreecommitdiff
path: root/.dev-suite/ticket/closed/ds-install-wipes-user-path-on-windows.toml
diff options
context:
space:
mode:
authorAntoine Martin <antoine97.martin@gmail.com>2020-01-05 00:32:56 +0100
committerAntoine Martin <antoine97.martin@gmail.com>2020-01-05 00:41:22 +0100
commitc8e11a8f9817bc780cb5858a946e5845788badd5 (patch)
tree0a5c22b77286cf2c1f6a4d5ce6608f0469dbd3cf /.dev-suite/ticket/closed/ds-install-wipes-user-path-on-windows.toml
parent2aef952141db26df131c201dbe672bce9eafc5d3 (diff)
Fix PATH getting wiped on Windows during install
There was no variable expansion called for %PATH% in setx PATH <dev-suite>;%PATH% effectively replacing PATH with litteraly `<dev-suite>;%PATH%`. The fix is simple, the command just needs to be called inside a shell with `cmd /C`.
Diffstat (limited to '.dev-suite/ticket/closed/ds-install-wipes-user-path-on-windows.toml')
-rw-r--r--.dev-suite/ticket/closed/ds-install-wipes-user-path-on-windows.toml24
1 files changed, 24 insertions, 0 deletions
diff --git a/.dev-suite/ticket/closed/ds-install-wipes-user-path-on-windows.toml b/.dev-suite/ticket/closed/ds-install-wipes-user-path-on-windows.toml
new file mode 100644
index 0000000..eb168e9
--- /dev/null
+++ b/.dev-suite/ticket/closed/ds-install-wipes-user-path-on-windows.toml
@@ -0,0 +1,24 @@
+title = 'ds install wipes user PATH on Windows'
+status = 'Closed'
+id = 'a2448280-2f41-11ea-b9b7-55d06c658ffe'
+assignees = []
+description = '''
+Currently on Windows, `ds install` calls the following command to add the folder
+containing downloaded dev-suite binaries to the PATH:
+
+```
+setx PATH <dev-suite location>;%PATH%
+```
+
+However this isn't called inside a shell, so variable expansion isn't performed,
+and the PATH the user had previously set is lost into oblivion.
+
+A simple fix is to call this instead:
+
+```
+cmd /C "setx PATH <dev-suite location>;%PATH%"
+```
+'''
+version = 'V1'
+
+[comments]