platformio / platformio-docs

PlatformIO Documentation
https://docs.platformio.org
Apache License 2.0
234 stars 323 forks source link

Fix windows Scripts path in shell-commands.rst #319

Closed caverar closed 9 months ago

caverar commented 9 months ago

Just a little fix to the path "C:\Users\UserName.platformio\penv\Scripts\" changing the ";" to "\" so the Windows shell is able to properly open all the scripts inside the folder.

CLAassistant commented 9 months ago

CLA assistant check
All committers have signed the CLA.

ivankravets commented 9 months ago

Should it be C:\Users\UserName\.platformio\penv\Scripts\;?

caverar commented 9 months ago

I modified the environment variable Path manually from the Windows UI and it worked only without the ;, something like this: C:\Users\UserName\.platformio\penv\Scripts\, I suppose that the ; was there for modifying the variable from the PowerShell (or just a typo?), but could be a bad idea putting it there because (from what I understood on stackoverflow) seems like there is multiple ways for changing the path variables with and without the ;, and I think the majority of Windows users is going to change the environment variable by the UI anyway.

ivankravets commented 9 months ago

How do you configure multiple directories to the PATH env?

caverar commented 9 months ago

In the Windows UI each env variable is a table, so I just add a new entry.

image

Anyway seems like there is too many ways to do it, so I think C:\Users\UserName\.platformio\penv\Scripts\ with the \ and without the ; is the more clear way for all the users.

ivankravets commented 9 months ago

Does it work without the \ at the end of the path? C:\Users\UserName\.platformio\penv\Scripts

caverar commented 9 months ago

In Windows 10 only worked for me with the \ at the end.

ivankravets commented 9 months ago

Thanks for the PR!

caverar commented 9 months ago

Thanks to you for Platformio, this sofware is great.