gh-155031: repass permission errors on subprocesses to parent / user - #155032
gh-155031: repass permission errors on subprocesses to parent / user#155032flowln wants to merge 2 commits into
Conversation
|
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase |
@picnixz I've tried implementing a test, though I didn't find a portable way to force a PermissionError on a child process, I though about doing |
These changes enable issues with permissions that happen while using the '--subprocesses' option on the 'run' method to propagate back, giving the user an indication of what went wrong. Previously, it would just silently fail, not giving the user a hint of what happened wrong. The `__main__` code was adjusted to streamline the `sys.exit` calls. Signed-off-by: Sofia Donato Ferreira <flowlnlnln@gmail.com>
Signed-off-by: Sofia Donato Ferreira <flowlnlnln@gmail.com>
7ae23fd to
620339d
Compare
picnixz
left a comment
picnixz
left a comment
There was a problem hiding this comment.
@pablogsal Is there a way to test this (maybe you can accept this as is)
2 participants
These changes enable issues with permissions that happen while using the
--subprocessesoption on the 'run' method to propagate back, giving the user an indication of what went wrong. Previously, it would just silently fail, not giving the user a hint of what happened wrong.