Change shebangs, add missing extension to check-dependencies #2

Merged
bigMouthCommie merged 2 commits from judy/storytime:shebang into main 2024-04-13 01:31:59 +02:00
Collaborator

🎁 Summary

Using #!/usr/bin/env bash instead of #!/bin/bash makes scripts more portable. bash isn't guaranteed to be in /bin/ on every system. /usr/bin/env uses the users's path, which should always find bash for us.

(Technically we could make it even more portable via #!/usr/bin/env sh, but I have not tested this with dash)

Quality Assurances

  • Works on my machine
🎁 Summary --- Using `#!/usr/bin/env bash` instead of `#!/bin/bash` makes scripts more portable. bash isn't guaranteed to be in /bin/ on every system. /usr/bin/env uses the users's path, which should always find bash for us. (Technically we could make it even *more* portable via #!/usr/bin/env sh, but I have not tested this with dash) ✅ Quality Assurances --- - [x] Works on my machine
judy force-pushed shebang from 111010077c to b6a02d14db 2024-04-12 20:08:45 +02:00 Compare
judy requested review from bigMouthCommie 2024-04-12 20:11:10 +02:00

I'm using the gitnex client and I have no idea what to do with your pull request.

I'm using the gitnex client and I have no idea what to do with your pull request.
bigMouthCommie merged commit 3b67804334 into main 2024-04-13 01:31:59 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: bigMouthCommie/storytime#2
No description provided.