MPAS-Dev / MPAS-Tools

MPAS Tools Repository
Other
37 stars 63 forks source link

Fix string `args` passed to `logging.check_call()` #461

Closed xylar closed 2 years ago

xylar commented 2 years ago

Previously, it assumed that args is a list but it now also supports string args that allow shell=True. This is useful for joining multiple commands into a single string.

xylar commented 2 years ago

Testing

I used this with this branch of compass: https://github.com/xylar/compass/tree/add_matrix_build With this functionality, I am able to run subprocesses with the shell=True argument.