ntpsec | Add dextral mode and srchost variable use options and better column autowidth. (!1033)

James Browning jamesb.fe80 at gmail.com
Sun Feb 23 02:57:05 UTC 2020


On Sat, Feb 22, 2020, at 6:40 PM Hal Murray <hmurray at megapathdsl.net> wrote:
>
> Looks like the second test is backwards.  It's printing the message on a
> system where pipefail works.
>
> if (set -o pipefail) 2>/dev/null
> then
>   echo "### Old sh - no pipefail"
>   echo "### We can't test for errors during build"
>   echo "### You will have to scan the log files."
>   PURGE="${PURGE} pipefail"
> fi

I am reasonably sure that is happening because I rewrote that test. It
should probably replace the first line with.

(set -o pipefail) 2>/dev/null
if [[ $? -ne 0 ]]

or something roughly equivalent to test the exit code not the contents
from stdout. I could be wrong a third time though.


More information about the devel mailing list