[darcs-users] darcs patch: add test that record doesn't read all ol... (and 1 more)

[Trent W. Buck]

Vincent Zweije <vzweije at zweije.nl.eu.org> writes:

> On Sun, May 03, 2009 at 04:04:13PM +1000, Trent W.Buck wrote:
>
> ||  +which strace || (echo skipping test since strace was not found; exit 0)
>
> Without trying it out: this test doesn't work.

Oops, I have no idea how that line got there, it should have just said
"which strace".  Did you somehow manage to only apply one of the two
patches in the bundle?

Due to set -e in ../tests/lib, "which strace" would DTRT for a bugs/ test.

> () introduces a subshell. exit 0 exits the subshell only. Using {} may
> work, as that doesn't (necessarily) introduce the subshell.

Yep, I know.  I've been thinking about adding set -E to tests/lib to
help with this.