Show exit codes in unified tests?
Matt Mackall
mpm at selenic.com
Wed Sep 15 19:57:46 CDT 2010
On Thu, 2010-09-16 at 00:30 +0200, Martin Geisler wrote:
> Dirkjan Ochtman <dirkjan at ochtman.nl> writes:
>
> > On Wed, Sep 15, 2010 at 22:14, Matt Mackall <mpm at selenic.com> wrote:
> >> This only show nonzero exit codes. Turning this on currently breaks 107
> >> tests. And there are probably issues with different exit codes from
> >> non-hg tools on different platforms we'd need to sort out too.
> >
> > I like the idea. Can we come up with a method to alleviate
> > cross-platform incompatibility? I.e. allowing specifying multiple
> > allowed return codes.
>
> I like it too -- and can we not catch the different error codes using a
> regexp like we do the for the rest of the test? Something like
> $ command
> \[(1|42|255)\]
That can be made to work in theory but the current proof of concept is a
bit too kludgey.
> though the escaping makes the square brackets a bad choice. Maybe like
> this instead
>
> $ command
> exit code: (1|42|255)
>
> which I think reads quite nicely.
Well what makes or breaks this idea is how big the delta is between
Solaris, BSD/OS X, and Linux return codes in our test suite. It may be
nearly negligible, in which case I'd like to keep the [1] style.
--
Mathematics is the supreme nostalgia of our time.
More information about the Mercurial-devel
mailing list