gvm icon indicating copy to clipboard operation
gvm copied to clipboard

Path incorrectly modified by GVM script

Open andrew-ttd opened this issue 11 months ago • 8 comments

After installing GVM the line below was added to my .zshrc:

[[ -s "/Users/andrew.oh/.gvm/scripts/gvm" ]] && source "/Users/andrew.oh/.gvm/scripts/gvm"

This script seems to incorrectly update my PATH environment variable by replacing spaces with :. For example:

> env

PATH=...:/Users/me/Library/Application:Support/JetBrains/Toolbox/scripts

GVM_PATH_BACKUP=...:/Users/me/Library/Application Support/JetBrains/Toolbox/scripts

andrew-ttd avatar Mar 04 '24 03:03 andrew-ttd

I'm seeing the same behaviour, but in this case it's mangling the path for VS Code:

In GVM_PATH_BACKUP: /mnt/c/Users/bearps/AppData/Local/Programs/Microsoft VS Code/bin

In PATH: /mnt/c/Users/bearps/AppData/Local/Programs/Microsoft:VS:Code/bin

The version I'm running:

$ gvm version
Go Version Manager v1.0.22 installed at /home/bevan/.gvm

theunrepentantgeek avatar Mar 07 '24 05:03 theunrepentantgeek

Same problem here, gvm is replacing spaces with :

caioreix avatar Mar 14 '24 01:03 caioreix

Ditto. I spent a day debugging to see why code does not work when called from the terminal on WSL. The conclusion is the same, colon (:) replaces spaces.

Also, it replaces cd as well.

(before gvm scripts mod)

❯ type cd
cd is a shell builtin

(after gvm scripts mod)

❯ type cd
cd is a shell function from /home/zlatko/.gvm/scripts/env/cd

zalom avatar Mar 27 '24 05:03 zalom

@zalom i found a solution, just the space from this path ~/.gvm/scripts/function/munge_path line 51. Something like this IFS=':' path_in_ary=( $(printf "%s" "${path_in}") ) IFS="$defaultIFS"

caioreix avatar Mar 27 '24 06:03 caioreix

@zalom i found a solution, just the space from this path ~/.gvm/scripts/function/munge_path line 51. Something like this IFS=':' path_in_ary=( $(printf "%s" "${path_in}") ) IFS="$defaultIFS"

Thanks man!

zalom avatar Mar 27 '24 06:03 zalom

This works for me. Thanks @caioreix !

Is this a typo or for some reason it is meant to be ': '? If its just a mistake maybe somebody can make a quick PR and fix it permanently?

Ivan-Parushev avatar May 21 '24 08:05 Ivan-Parushev

Installed gvm today and had the same problem. Windows 10 WSL2 (Ubuntu 22.04) Go Version Manager v1.0.22

Removing the extra space in line 51 of ~/.gvm/scripts/function/munge_path does not work for me.

Also further to what was already reported (replacement of spaces with a colon), Looking closely at the changes made to the PATH. I can see that many of my paths were simply chopped off. Here are some examples

FROM: /mnt/c/Program Files/Eclipse Adoptium/jdk-8.0.382.5-hotspot/bin /mnt/c/Program Files/nodejs

TO: Adoptium/jdk-8.0.382.5-hotspot/bin Files/nodejs

So the problem is not limited with replacing spaces with colon. Some paths simply get chopped off at a space.

kaxxa123 avatar Jul 17 '24 18:07 kaxxa123

@kaxxa123 I think this could be the solution. https://github.com/moovweb/gvm/issues/486#issuecomment-2273699373

Heunsig avatar Aug 07 '24 15:08 Heunsig