[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).
From: |
Stefano Lattarini |
Subject: |
Re: [PATCH] Enable `set -e' in more tests (plus some tweakings). |
Date: |
Sun, 22 Aug 2010 16:12:04 +0200 |
User-agent: |
KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; ) |
On Sunday 22 August 2010, Ralf Wildenhues wrote:
> Hi Stefano,
>
> * Stefano Lattarini wrote on Sun, Aug 22, 2010 at 03:10:54PM CEST:
> > This patch shouldn't cause problems, as the modified tests do not
> > use shell functions nor complex shell constructs, loops or
> > conditionals (with the exception of `info.test', but see the
> > "FIXME" comment added to it).
>
> Please state if and how you tested the 'set -e' patches (e.g.,
> Solaris shell, heirloom shell, etc). That helps immensely for
> reviewing them.
I'll do that soonish (not immediately, though). Sorry for not having
thought of that myself.
Stefano