Skip to content
Snippets Groups Projects
Commit 2f6e45ca authored by Sami Kerola's avatar Sami Kerola Committed by Kamil Dudka
Browse files

test: fix output redirection [shellcheck]

SC2069: The order of the 2>&1 and the redirect matters.  The 2>&1 has to be
last.

Maintainer propsed stderr should be visible, see review link for details.

Reference; https://github.com/koalaman/shellcheck/wiki/SC2069
Review: https://github.com/logrotate/logrotate/pull/138


Signed-off-by: default avatarSami Kerola <kerolasa@iki.fi>

Closes #138
parent 1aac849e
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -143,7 +143,7 @@ checkoutput() {
echo expected: \'$expected\'
exit 2
fi
echo "$config_crc" | md5sum -c - 2>&1 > /dev/null
echo "$config_crc" | md5sum -c - >/dev/null
if [ $? != 0 ]; then
echo "config file $output has been altered: MD5 sum mismatch"
exit 3
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment