Hi wisecracker,
Thank you for catching the mismatched double-quotes. I was trying to change "$var"/string to "$var/string" and didn't complete the job in a couple of lines.

I'm sorry that shellcheck doesn't like stand-alone redirections. They are part of the shell command language and there is no need to combine them with another command when used as they are used in the script being discussed in this thread.

Note also that the commands:

Code:

true > "${LOG_DIR}/message"
true > "${LOG_DIR}/wtmp"

produces the same results in the script being discussed in this thread but runs slower and just like:

Code:

> "${LOG_DIR}/message"
> "${LOG_DIR}/wtmp"

will give a zero exit status even when > "${LOG_DIR}/message" failed if > "${LOG_DIR}/wtmp" succeeded.

Of course, the discussion would be different if other pipelines that send output to standard output without redirecting that output were to be included in the script after these redirections; but that isn't the case here in cleanup2. Nonetheless, I should have stated this assumption in the example I provided.