At Sunday 11 April 2010, Ralf Wildenhues <[email protected]>
wrote:
> Hello Stefano,
>
> * Stefano Lattarini wrote on Fri, Apr 09, 2010 at 07:25:31PM CEST:
> > -$ACLOCAL || Exit 1
> > -$AUTOMAKE || Exit 1
> > +$ACLOCAL
> > +$AUTOMAKE
>
> there is no need to actually remove these.
Strictly speaking, no; but their removal would make the script more
consistent with the great majority of the other tests which use
`set -e'.
Anyway, I'm fine with keeping those `|| Exit', since the "consistency"
I'm speaking about here is purely cosmetic, bordering on pointless
nitpicking (on the contrary, the use `set -e' concerns IMO semantic
consistency, which is far far more important).
> They may not be
> strictly required any more with 'set -e', but they may still serve
> as visual reminder that this is an important part of the test.
Thanks,
Stefano
