search

dsa-ou / m269-installer

Software installation script and instructions for M269
https://dsa-ou.github.io/m269-installer/
BSD 3-Clause "New" or "Revised" License
0 stars 1 forks source link

Space in username on windows causes aliases to fail #27

Closed densnow closed 1 year ago

densnow commented 1 year ago

The title pretty much says it all. This could possibly be fixed by changing line 98 in install.sh to:

& '$VENV\Scripts\Activate.ps1'
densnow commented 1 year ago

We might need to add quotes to any lines that use $VENV, namely 83 and 86

densnow commented 1 year ago

After minimal testing it looks like only line 98 needs to be changed. When $VENV is used as arg or with . (dot) in front of path it seems to be protected against spaces in the path.

mwermelinger commented 1 year ago

Thanks, please do a PR.