fix: Added quotes to unquoted paths in python install and pip upgrade commands - #378
Conversation
Sorry, something went wrong.
There was a problem hiding this comment.
Pull request overview
Updates the Windows Python toolcache setup PowerShell template to better handle hosted tool paths that contain spaces, specifically during pip bootstrapping/upgrade.
Changes:
- Quote $PythonExePath within the cmd.exe /c invocation for ensurepip and pip install --upgrade to avoid failures when the Python path contains spaces.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
|
Hello all, simple change that likely affects a lot of consumers. Politely requesting a review from team? |
Sorry, something went wrong.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
Added backtick commented out quotes for python install and pip upgrade steps to avoid errors when the hosted tools path has a space in it.