Skip to content

Restore execute bits on deactivate scripts - #23620

Merged
Karthik Nadig (karthiknadig) merged 1 commit into
microsoft:mainfrom
covracer:chmod+x
Jun 17, 2024
Merged

Restore execute bits on deactivate scripts#23620
Karthik Nadig (karthiknadig) merged 1 commit into
microsoft:mainfrom
covracer:chmod+x

Conversation

@covracer

@covracer Christopher Covington (covracer) commented Jun 14, 2024

Copy link
Copy Markdown

#22921 removed executable bits:

diff --git a/pythonFiles/deactivate/bash/deactivate b/python_files/deactivate/bash/deactivate
old mode 100755
new mode 100644
similarity index 100%
rename from pythonFiles/deactivate/bash/deactivate
rename to python_files/deactivate/bash/deactivate

https://github.com/microsoft/vscode-python/pull/22921/files#diff-796809259ce3b33f54a33371e898048faf2f7f912cd1bbe11059feb40a63a58d

Set them back.

Fixes #23449 and Fixes #23195

@covracer Christopher Covington (covracer) changed the title Set executable bit on deactivate scripts Restore execute bits on deactivate scripts Jun 14, 2024
@covracer

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@anthonykim1

Copy link
Copy Markdown

Thank you for doing this

@karthiknadig
Karthik Nadig (karthiknadig) merged commit c066a3f into microsoft:main Jun 17, 2024
Don Jayamanne (DonJayamanne) pushed a commit that referenced this pull request Jun 24, 2024
#22921 removed executable bits:
```diff
diff --git a/pythonFiles/deactivate/bash/deactivate b/python_files/deactivate/bash/deactivate
old mode 100755
new mode 100644
similarity index 100%
rename from pythonFiles/deactivate/bash/deactivate
rename to python_files/deactivate/bash/deactivate
```


https://github.com/microsoft/vscode-python/pull/22921/files#diff-796809259ce3b33f54a33371e898048faf2f7f912cd1bbe11059feb40a63a58d

Set them back.


Fixes #23449 and #23195
Eleanor Boyd (eleanorjboyd) pushed a commit to eleanorjboyd/vscode-python that referenced this pull request Jun 28, 2024
microsoft#22921 removed executable bits:
```diff
diff --git a/pythonFiles/deactivate/bash/deactivate b/python_files/deactivate/bash/deactivate
old mode 100755
new mode 100644
similarity index 100%
rename from pythonFiles/deactivate/bash/deactivate
rename to python_files/deactivate/bash/deactivate
```


https://github.com/microsoft/vscode-python/pull/22921/files#diff-796809259ce3b33f54a33371e898048faf2f7f912cd1bbe11059feb40a63a58d

Set them back.


Fixes microsoft#23449 and microsoft#23195
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issue identified by VS Code Team member as probable bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot deactivate venv in terminal while venv/bin/python is interpreter Deactivate Script in remote server is Permission denied

3 participants