diff options
author | Brian Behlendorf <[email protected]> | 2018-01-17 10:17:16 -0800 |
---|---|---|
committer | GitHub <[email protected]> | 2018-01-17 10:17:16 -0800 |
commit | 3da3488e6339ff2dc5c7f3da8c8a0c552d018d68 (patch) | |
tree | acf9843a79221a7fa9de852507d3cdef8772ee44 /scripts/zimport.sh | |
parent | d658b2caa95726c13d99123874910cdedc7ce866 (diff) |
Fix shellcheck v0.4.6 warnings
Resolve new warnings reported after upgrading to shellcheck
version 0.4.6. This patch contains no functional changes.
* egrep is non-standard and deprecated. Use grep -E instead. [SC2196]
* Check exit code directly with e.g. 'if mycmd;', not indirectly
with $?. [SC2181] Suppressed.
Reviewed-by: George Melikov <[email protected]>
Reviewed-by: Giuseppe Di Natale <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #7040
Diffstat (limited to 'scripts/zimport.sh')
-rwxr-xr-x | scripts/zimport.sh | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/scripts/zimport.sh b/scripts/zimport.sh index 36d4e6696..61c9aba4a 100755 --- a/scripts/zimport.sh +++ b/scripts/zimport.sh @@ -566,6 +566,7 @@ for TAG in $POOL_TAGS; do $ZPOOL_CMD import -N -d "$POOL_DIR_COPY" \ "$POOL_NAME" &>/dev/null + # shellcheck disable=SC2181 if [ $? -ne 0 ]; then fail_nonewline ERROR=1 |