-
Notifications
You must be signed in to change notification settings - Fork 25
Evaluate using Popen(shell=False, ...) #250
Copy link
Copy link
Open
Labels
good first issuea relatively simple task to familiarize oneself with the code base, and the contribution processa relatively simple task to familiarize oneself with the code base, and the contribution processhelp wantedWe'd like help with this issue, and are happy to provide guidanceWe'd like help with this issue, and are happy to provide guidance
Description
Metadata
Metadata
Assignees
Labels
good first issuea relatively simple task to familiarize oneself with the code base, and the contribution processa relatively simple task to familiarize oneself with the code base, and the contribution processhelp wantedWe'd like help with this issue, and are happy to provide guidanceWe'd like help with this issue, and are happy to provide guidance
Should we avoid using the shell for running commands?
Think this is a major change at the core of ReBench.
So, we'd probably need to carefully test this.
See #248 (comment)