search

neogeographica / chaintool

WIP
GNU General Public License v3.0
0 stars 0 forks source link

should be able to specify shell used for cmd run #12

Open neogeographica opened 3 years ago

neogeographica commented 3 years ago

(via subprocess.call... currently uses the default executable /bin/sh)

Ideally this would be specifiable on a per-cmd basis, since command syntax might depend on the shell. However there's currently no way to attach extraneous configuration info to a cmd.

neogeographica commented 3 years ago

A less satisfying but still consistent "solution" would be to NOT specify shell=True for subprocess.call. If someone needs to do script-y things they can write a script, and then invoke that script as a cmd. I'd have to shlex.split the command.