mirror of https://github.com/bitcoin/bitcoin.git
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).
This commit is contained in:
parent
faf65f0531
commit
7777fb8bc7
|
@ -1123,27 +1123,7 @@ TRY_LOCK(cs_vNodes, lockNodes);
|
|||
Scripts
|
||||
--------------------------
|
||||
|
||||
Write scripts in Python rather than bash, when possible.
|
||||
|
||||
### Shebang
|
||||
|
||||
- Use `#!/usr/bin/env bash` instead of obsolete `#!/bin/bash`.
|
||||
|
||||
- [*Rationale*](https://github.com/dylanaraps/pure-bash-bible#shebang):
|
||||
|
||||
`#!/bin/bash` assumes it is always installed to /bin/ which can cause issues;
|
||||
|
||||
`#!/usr/bin/env bash` searches the user's PATH to find the bash binary.
|
||||
|
||||
OK:
|
||||
```bash
|
||||
#!/usr/bin/env bash
|
||||
```
|
||||
|
||||
Wrong:
|
||||
```bash
|
||||
#!/bin/bash
|
||||
```
|
||||
Write scripts in Python or Rust rather than bash, when possible.
|
||||
|
||||
Source code organization
|
||||
--------------------------
|
||||
|
|
|
@ -26,6 +26,9 @@ ALLOWED_SOURCE_FILENAME_EXCEPTION_REGEXP = (
|
|||
ALLOWED_PERMISSION_NON_EXECUTABLES = 0o644
|
||||
ALLOWED_PERMISSION_EXECUTABLES = 0o755
|
||||
ALLOWED_EXECUTABLE_SHEBANG = {
|
||||
# https://github.com/dylanaraps/pure-bash-bible#shebang:
|
||||
# `#!/bin/bash` assumes it is always installed to /bin/ which can cause issues;
|
||||
# `#!/usr/bin/env bash` searches the user's PATH to find the bash binary.
|
||||
"py": [b"#!/usr/bin/env python3"],
|
||||
"sh": [b"#!/usr/bin/env bash", b"#!/bin/sh"],
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue