CABLE-LSM / benchcab

Tool for evaluation of CABLE land surface model
https://benchcab.readthedocs.io/en/latest/
Apache License 2.0
2 stars 4 forks source link

Abstract shell calls, add shlex #146

Open bschroeter opened 1 year ago

bschroeter commented 1 year ago

It would be good to have a single point to test with automatic shell command escaping.

The logic here is to have a single interface to communicate with the shell, rather that recreating the subprocess object each time.

The function would take a command with arguments, automatically shell-escaping the arguments with shlex (protecting from shell injection). Any additional kwargs will be passed directly to subprocess to override the default arguments.

This could be a common function somewhere?

SeanBryan51 commented 1 year ago

Hey team! Please add your planning poker estimate with Zenhub @bschroeter @ccarouge