rancher / dapper

Docker build wrapper
Apache License 2.0
365 stars 89 forks source link

Let simple scripts use /bin/sh, and handle whitespace paths #75

Closed joakimr-axis closed 4 years ago

joakimr-axis commented 4 years ago

The scripts here are very lightweight and simple, and do not need or use the wide range of extra functionality provided by bash over what is found in a small and fast POSIX shell (like e.g. dash). Hence those scripts can rely on /bin/sh instead of /bin/bash.

Furthermore, this patch adds quotes for the dirname in the scripts, in case there are whitespaces in the paths.