racket / docker

Docker images for various Racket versions
MIT License
62 stars 17 forks source link

build(foreach): handle arguments robustly #78

Closed benknoble closed 5 months ago

benknoble commented 5 months ago

Fix some shellcheck warnings and, more importantly, handle the arguments more robustly by not word-splitting arguments. An invocation like

foreach F "a b" c

would have run F a; F b; F c. Make it run F "a b"; F c as expected.

benknoble commented 5 months ago

I see that @jackfirth did something similar in https://github.com/racket/docker/pull/63; I elected to keep the separate arg extraction for readability (I usually write local, though).