7x31/shellcheck
1
0
Fork 0
mirror of https://github.com/koalaman/shellcheck.git synced 2025-10-03 19:29:44 +08:00
Code Issues Packages Projects Releases Wiki Activity

Copyedited, added links to POSIX

Lawrence Velázquez
2024-03-08 19:43:55 -05:00
parent 9798448ea8
commit 6708aaa197
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
SC3016.md

@@ -12,8 +12,7 @@ fi
### Correct code: ### Correct code:
Either switch to bash: Either switch to bash or ksh, or use:
```sh ```sh
#!/bin/sh #!/bin/sh
if [ -n "${STY+x}" ] if [ -n "${STY+x}" ]
@@ -26,7 +25,7 @@ fi
Your script uses a shell feature not supported by the shebang. Either rewrite the script to be portable, or change the shebang to explicitly require a shell like Bash. Your script uses a shell feature not supported by the shebang. Either rewrite the script to be portable, or change the shebang to explicitly require a shell like Bash.
In this case, `[ -v variable ]` to check if a variable is set can be replaced with `[ -n "${variable+x}" ]` which uses the "alternative value if set" parameter expansion syntax to accomplish the same thing. In this case, `[ -v variable ]`, which checks if a variable is set, is [not specified by POSIX](https://pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html). It can be replaced with the portable `[ -n "${variable+x}" ]`, which uses the "alternative value if set" [parameter expansion](https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_06_02) syntax to accomplish the same thing.
### Exceptions: ### Exceptions: