Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Please use shutil.which() instead of external which program #333

Closed
mgorny opened this issue May 13, 2023 · 1 comment · Fixed by #346
Closed

Please use shutil.which() instead of external which program #333

mgorny opened this issue May 13, 2023 · 1 comment · Fixed by #346

Comments

@mgorny
Copy link

mgorny commented May 13, 2023

The which program is being phased out and is no longer installed on Gentoo systems by default. Please use shutil.which() instead.
gentoo-bot pushed a commit to gentoo/gentoo that referenced this issue Jun 4, 2023
@thesamesam
dev-python/nodeenv: add link to upstream bug re which
c3ec645 
Bug: ekalinin/nodeenv#333
Signed-off-by: Sam James <sam@gentoo.org>
thesamesam added a commit to thesamesam/nodeenv that referenced this issue Dec 13, 2023
@thesamesam
Remove usage of non-portable which
e648570 
* Use Python's shutil.which() instead of shelling out to `which` to find Python 2
* Use `command -v` instead of `which` in README

Fixes: ekalinin#333
@thesamesam thesamesam mentioned this issue Dec 13, 2023
Merged
thesamesam added a commit to thesamesam/nodeenv that referenced this issue Dec 13, 2023
@thesamesam
Remove usage of non-portable which
0d3ebaf 
* Use Python's shutil.which() instead of shelling out to `which` to find Python 2
* Use `command -v` instead of `which` in README

Fixes: ekalinin#333
gentoo-bot pushed a commit to gentoo/gentoo that referenced this issue Dec 13, 2023
@thesamesam
dev-python/nodeenv: drop which dependency
6fec13b 
Bug: ekalinin/nodeenv#333
Signed-off-by: Sam James <sam@gentoo.org>
ekalinin pushed a commit that referenced this issue May 28, 2024
@thesamesam
Remove usage of non-portable which (#346)
1024f4f 
* Use Python's shutil.which() instead of shelling out to `which` to find Python 2
* Use `command -v` instead of `which` in README

Fixes: #333
@mgorny
Copy link
Author

mgorny commented May 28, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove usage of non-portable which thesamesam/nodeenv
1 participant
@mgorny