Akim Demaille <[EMAIL PROTECTED]> writes:
>>>> "Paul" == Paul Eggert <[EMAIL PROTECTED]> writes:
>
> > Akim Demaille <[EMAIL PROTECTED]> writes:
> >> -set +e
> >> +test -z "$VERBOSE" && {
> >> + exec > /dev/null 2>&1
> >> +}
>
> > Don't you need to keep the "set +e"? As I recall, some shells send
> > diagnostics to the original stderr, even if you've redirected it with
> > 'exec' in the mean time.
>
> I don't see the connection. Aren't you confused with set +x?
Yes I was, sorry.
I was confused because the ChangeLog entry talks only about being
quiet, and yet the patch removed the "set +e". Shouldn't the removal
of "set +e" have been logged as well?
Or perhaps the "set +e" should have been kept? Presumably it was
there for a reason. E.g.:
set +e
case $VERBOSE in
'') exec > /dev/null 2>&1 ;;
esac
