eth-cscs / DLA-Future

DLA-Future
https://eth-cscs.github.io/DLA-Future/master/
BSD 3-Clause "New" or "Revised" License
64 stars 14 forks source link

Remove `eval` from wrapper scripts #1093

Closed msimberg closed 8 months ago

msimberg commented 8 months ago

Don't use eval to avoid surprising variable expansion. Variable expansion can instead be opted into by explicitly wrapping a command in eval or bash -c.