From de9a8869a8829a59ebda47a820a99fa9d9f41978 Mon Sep 17 00:00:00 2001 From: Vidar Holen Date: Wed, 2 Nov 2022 20:51:41 -0700 Subject: [PATCH] Created SC3016 (markdown) --- SC3016.md | 37 +++++++++++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+) create mode 100644 SC3016.md diff --git a/SC3016.md b/SC3016.md new file mode 100644 index 0000000..074525c --- /dev/null +++ b/SC3016.md @@ -0,0 +1,37 @@ +## In POSIX sh, unary `-v` (in place of `[ -n "${var+x}" ]`) is undefined. + +### Problematic code: + +```sh +#!/bin/sh +if [ -v STY ] +then + echo "STY is set, you are using screen" +fi +``` + +### Correct code: + +Either switch to bash: + +```sh +#!/bin/sh +if [ -n "${STY+x}" ] +then + echo "STY is set, you are using screen" +fi +``` + +### Rationale: + +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. + +### Exceptions: + +None + +### Related resources: + +* Help by adding links to BashFAQ, StackOverflow, man pages, POSIX, etc! \ No newline at end of file