From ea4ead2bab5cef0f8c07d65d0b957db12a2c334b Mon Sep 17 00:00:00 2001 From: Vidar Holen Date: Tue, 1 Sep 2020 18:08:12 -0700 Subject: [PATCH] Created SC3017 (markdown) --- SC3017.md | 39 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+) create mode 100644 SC3017.md diff --git a/SC3017.md b/SC3017.md new file mode 100644 index 0000000..4d663b3 --- /dev/null +++ b/SC3017.md @@ -0,0 +1,39 @@ +## In POSIX sh, unary `-a` in place of `-e` is undefined. + +(or "In dash, ... is not supported." when using `dash`) + +### Problematic code: + +```sh +#!/bin/sh +if [ -a file.txt ] +then + echo "file.txt exists" +fi +``` + +### Correct code: + +```sh +#!/bin/sh +if [ -e file.txt ] +then + echo "file.txt exists" +fi +``` + +### Rationale: + +`-a` was used in some shells to check for the existence of a file, but the POSIX standard and better supported equivalent is `-e`. + +### Exceptions: + +If you only intend to target shells that supports this feature, you can change +the shebang to a shell that guarantees support, or [[ignore]] this warning. + +You can use `# shellcheck disable=SC3000-SC4000` to ignore all such compatibility +warnings. + +### Related resources: + +* Help by adding links to BashFAQ, StackOverflow, man pages, POSIX, etc!