faasm icon indicating copy to clipboard operation
faasm copied to clipboard
verified publisher icon faasm

Make scripts zsh proof

Open mmathys opened this issue 3 years ago • 2 comments

E.g. workon.sh is zsh proof, but not create_venv.sh. bash and zsh evaluate the expression for THIS_DIR differently.

mmathys avatar Aug 30 '22 20:08 mmathys

See https://github.com/auto-ndp/faabric/blob/master/bin/workon.sh#L10, maybe I should PR this upstream :sweat_smile:

eigenraven avatar Aug 31 '22 11:08 eigenraven

That solution looks clean! Yeah this may be a good idea :)

mmathys avatar Aug 31 '22 12:08 mmathys