pypa / pipx

Install and Run Python Applications in Isolated Environments
https://pipx.pypa.io
MIT License
9.51k stars 396 forks source link

Introduce a new env variable to allow spaces in home path #1446

Closed Gitznik closed 2 weeks ago

Gitznik commented 3 weeks ago

Summary of changes

Introduces an environment variable to silence the warning for having a space in the pipx home path.

Improves the docs on how to troubleshoot the warning and move the pipx installation location in general. This makes the discussion I created a while ago redundant.

Resolves #1320

Test plan

Tested by running

nox -s tests-3.8 -- -k test_environment
Gitznik commented 3 weeks ago

@huxuan if you have the time, could you please verify that the windows example for moving the pipx install location actually works? Or maybe even translate it to powershell?

huxuan commented 3 weeks ago

@huxuan if you have the time, could you please verify that the windows example for moving the pipx install location actually works? Or maybe even translate it to powershell?

Gotcha, will take a look later.

huxuan commented 3 weeks ago

Could you please verify that the windows example for moving the pipx install location actually works? Or maybe even translate it to powershell?

Verified and translated! PTAL

Gitznik commented 2 weeks ago

Sorry had to add just one small change to make sure spaces in the paths are not a problem when using bash.

huxuan commented 2 weeks ago

Sorry had to add just one small change to make sure spaces in the paths are not a problem when using bash.

looks like the double quotes are also needed for the dirs and $NEW_LOCATION in powershell to work with spaces in paths.