AtomGraph / LinkedDataHub

The low-code Knowledge Graph application platform. Apache license.
https://atomgraph.github.io/LinkedDataHub/
Apache License 2.0
493 stars 124 forks source link

Use "/usr/bin/env bash" as interpreter in scripts #80

Closed tmciver closed 2 years ago

tmciver commented 3 years ago

It would be convenient to use the #!/usr/bin/env bash interpreter line at the top of all scripts so that they will work on systems where bash is on a different path.

My motivation for requesting this change is so that user scripts will work when run using Nix. Such user scripts typically leverage scripts in LDH and bash is not found under /bin when using Nix. (I ran into this problem while developing ldh-quotes app.)

namedgraph commented 2 years ago

@tmciver I'll try this on the develop branch. If it works, then it's fine with me to merge #81.

namedgraph commented 2 years ago

@tmciver this is now merged to develop. Wanna check whether it works for you? Will try to do a new release ASAP.

tmciver commented 2 years ago

Yup, it works! Thanks!