Skip to content

Commit

Permalink
[scrooge]: Portable shebang
Browse files Browse the repository at this point in the history
Problem:

/bin/bash is not available on non-FHS distros, such as NixOS

Solution:

Replace /bin/bash with /usr/bin/env bash

Result:

Closes #366

Differential Revision: https://phabricator.twitter.biz/D1107604
  • Loading branch information
andreoss authored and jenkins committed Oct 31, 2023
1 parent fe8a81d commit 28f450c
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion pushsite.bash
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/bin/bash
#!/usr/bin/env bash

set -e

Expand Down
2 changes: 1 addition & 1 deletion sbt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/bin/bash
#!/usr/bin/env bash

set -eo pipefail

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/bin/bash
#!/usr/bin/env bash

set -euxo pipefail

Expand Down
2 changes: 1 addition & 1 deletion scrooge-linter/src/scripts/linter-test
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/bin/bash
#!/usr/bin/env bash

# Run from source root
# e.g. ./pants bundle scrooge/scrooge-linter:app \
Expand Down

0 comments on commit 28f450c

Please sign in to comment.