Skip to content

Execute extra commands without shell=True#51

Open
borysiasty wants to merge 1 commit intomasterfrom
subprocess-fix
Open

Execute extra commands without shell=True#51
borysiasty wants to merge 1 commit intomasterfrom
subprocess-fix

Conversation

@borysiasty
Copy link
Owner

No description provided.

@borysiasty
Copy link
Owner Author

@ricardogsilva does this approach work for your use case? Instead of using shell=True, the commands are now executed line-by-line and arguments are parsed with shlex.split().

Note that this means shell features such as pipeline |, redirection >, substitution $(), chaining && etc. are no longer supported...

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant