From 2f1d10ba5d537611b22ceb2e7470438d61135fb2 Mon Sep 17 00:00:00 2001 From: Vidar Holen Date: Tue, 1 Sep 2020 17:52:05 -0700 Subject: [PATCH] Created SC3039 (markdown) --- SC3039.md | 35 +++++++++++++++++++++++++++++++++++ 1 file changed, 35 insertions(+) create mode 100644 SC3039.md diff --git a/SC3039.md b/SC3039.md new file mode 100644 index 0000000..1a05d4b --- /dev/null +++ b/SC3039.md @@ -0,0 +1,35 @@ +## In POSIX sh, 'let' is undefined. + +(or "In dash, ... is not supported." when using `dash`) + +### Problematic code: + +```sh +#!/bin/sh +let x=2+2 +``` + +### Correct code: + +Use standard `$((arithmetic expansion))` instead of legacy `let`: + +```sh +#!/bin/sh +x=$((2+2)) +``` + +### Rationale: + +`let` is a command used for arithmetic in some shells, but it's not a standard command. Use `$((..))` instead. + +### 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!