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.

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

twisted.test.test_process.PosixProcessPTYTests.test_executionError fails in some environments #10300

Closed
twisted-trac opened this issue Jan 28, 2022 · 5 comments

Comments

@twisted-trac
Copy link
Collaborator

@twisted-trac twisted-trac commented Jan 28, 2022

exarkun's avatar @exarkun reported
Trac ID trac#10300
Type defect
Created 2022-01-28 21:20:36Z

PosixProcessBase.getCommand only looks at /bin and /usr/bin for the requested command, ignoring PATH. There is a perfectly good helper for searching PATH easily at hand, procutils.which.

Searchable metadata
trac-id__10300 10300
type__defect defect
reporter__exarkun exarkun
priority__normal normal
milestone__None None
branch__ 
branch_author__ 
status__closed closed
resolution__fixed fixed
component__core core
keywords__None None
time__1643404836460727 1643404836460727
changetime__1645653623928084 1645653623928084
version__None None
owner__exarkun exarkun

@twisted-trac
Copy link
Collaborator Author

@twisted-trac twisted-trac commented Jan 28, 2022

exarkun's avatar @exarkun commented

#1689

@twisted-trac
Copy link
Collaborator Author

@twisted-trac twisted-trac commented Jan 28, 2022

wsanchez's avatar @wsanchez set owner to @exarkun

@twisted-trac
Copy link
Collaborator Author

@twisted-trac twisted-trac commented Jan 28, 2022

wsanchez's avatar @wsanchez commented

reviewed

@twisted-trac
Copy link
Collaborator Author

@twisted-trac twisted-trac commented Jan 31, 2022

exarkun's avatar @exarkun commented

Fix merged in 171fd5c

@twisted-trac
Copy link
Collaborator Author

@twisted-trac twisted-trac commented Feb 23, 2022

exarkun's avatar @exarkun set status to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants