Skip to content

Commit dcd2349

Browse files
MarcoFalkeRandyMcMillan
authored andcommitted
doc: Remove shebang section
This is already checked by test/lint/lint-files.py There is no need to reword all linters into the dev notes. Also, allow scripts in Rust (there are already some).
1 parent 6b0a349 commit dcd2349

File tree

2 files changed

+4
-21
lines changed

2 files changed

+4
-21
lines changed

doc/developer-notes.md

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1122,27 +1122,7 @@ TRY_LOCK(cs_vNodes, lockNodes);
11221122
Scripts
11231123
--------------------------
11241124
1125-
Write scripts in Python rather than bash, when possible.
1126-
1127-
### Shebang
1128-
1129-
- Use `#!/usr/bin/env bash` instead of obsolete `#!/bin/bash`.
1130-
1131-
- [*Rationale*](https://github.com/dylanaraps/pure-bash-bible#shebang):
1132-
1133-
`#!/bin/bash` assumes it is always installed to /bin/ which can cause issues;
1134-
1135-
`#!/usr/bin/env bash` searches the user's PATH to find the bash binary.
1136-
1137-
OK:
1138-
```bash
1139-
#!/usr/bin/env bash
1140-
```
1141-
1142-
Wrong:
1143-
```bash
1144-
#!/bin/bash
1145-
```
1125+
Write scripts in Python or Rust rather than bash, when possible.
11461126
11471127
Source code organization
11481128
--------------------------

test/lint/lint-files.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,9 @@
2626
ALLOWED_PERMISSION_NON_EXECUTABLES = 0o644
2727
ALLOWED_PERMISSION_EXECUTABLES = 0o755
2828
ALLOWED_EXECUTABLE_SHEBANG = {
29+
# https://github.com/dylanaraps/pure-bash-bible#shebang:
30+
# `#!/bin/bash` assumes it is always installed to /bin/ which can cause issues;
31+
# `#!/usr/bin/env bash` searches the user's PATH to find the bash binary.
2932
"py": [b"#!/usr/bin/env python3"],
3033
"sh": [b"#!/usr/bin/env bash", b"#!/bin/sh"],
3134
}

0 commit comments

Comments
 (0)