From: David Kastrup Date: Wed, 18 Jul 2007 13:27:12 GMT Subject: Re: [PATCH] filter-branch: get rid of "set -e" Message-ID: <86vechj0zz.fsf@lola.quinscape.zz> In-Reply-To: Johannes Schindelin writes: > It was reported by Alex Riesen that "set -e" can break something as > trivial as "unset CDPATH" in bash. Not in bash. Just tried it. But there are Bourne shells that do get annoyed. > So get rid of "set -e". I'd rather write [ "X$CDPATH" = "X" ] || unset CDPATH With regard to shells, there is an interesting document (info "(autoconf) Portable Shell") which is rather disillusioning. A real horror show is (info "(autoconf) Limitations of Builtins") where one learns that using if without else can lead to bad exit status, too. -- David Kastrup