Re: [PATCH] filter-branch: get rid of "set -e"

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

To: <git@...>

Subject: Re: [PATCH] filter-branch: get rid of "set -e"

Date: Wednesday, July 18, 2007 - 9:27 am

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

quoted text> 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.

quoted text> 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

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:

Re: [PATCH] filter-branch: get rid of "set -e", David Kastrup, (Wed Jul 18, 9:27 am)

Re: [PATCH] filter-branch: get rid of "set -e", Johannes Schindelin, (Wed Jul 18, 10:05 am)


linux-kernel:
Greg KH	Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3
Greg Kroah-Hartman	[PATCH 001/196] Chinese: Add the known_regression URI to the HOWTO
Andy Whitcroft	clam
Ingo Molnar	[patch] paravirt: VDSO page is essential
git:

linux-netdev:
David Miller	Re: [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock().
Gerrit Renker	[PATCH 27/37] dccp: Integration of dynamic feature activation - part 2 (server side)
Lovich, Vitali	RE: [PATCH] Packet socket: mmapped IO: PACKET_TX_RING
David Miller	[GIT]: Networking
openbsd-misc:

Re: [PATCH] filter-branch: get rid of "set -e"

Online users