Re: [PATCH] {master} tests: enable 'errexit' shell flag by default.

[Stefano Lattarini]

On Tuesday 11 January 2011, Ralf Wildenhues wrote:
> Hi Stefano,
> 
> * Stefano Lattarini wrote on Tue, Jan 11, 2011 at 03:02:09AM CET:
> > Finally, here is the patch that moves the setting of the `errexit'
> > shell flag in the `tests/defs' script.
> 
> > Here are the steps I followed to write the patch:
> > 
> >  [1] I got the list of non-generated test scripts:
> >       $ tests=`grep -L '^#.* GENERATED AUTOMATICALLY' tests/*.test`
> > 
> >  [2] I checked that each of those test scripts had *one only* occurence
> >      of a `set -e' line, and that this line occurred immediately after
> >      the inclusion the `./defs' script:
> >        $ perl set-e.pl $tests
> >      (see attachement for the set-e.pl script)
> > 
> >  [3] I used GNU sed to remove that `set -e' line and the single following
> >      blank line (if any) for each of those scripts:
> >       $ sed -i '/^set -e$/{ N; /^set -e\n *$/d; s/^set -e\n//; }' $tests
> > 
> >  [4] I updated files ChangeLog, tests/defs and tests/README by hand.
> > 
> > The patch is attached (compressed, as it's pretty big and repetitive).
> > Inline below are the most relevant hunks.
> 
> So you're saying we already use 'set -e' in every test? 
>
Yep.

> Nice.
> 
> Remains only bumping copyrights.
>
Hmm... Honestly, I left that out deliberately, since the edits done by
this patch to the test scripts are very obvious, and because I usually
take the copyright years to be a rough indicator of how long a file
hasn't been touched by "real" edits (I know the git history is more
faithful on this, but still).

> For that, I guess since the testsuite
> is going to be churned completely anyway, I suggest we just use the
> update-copyright script from gnulib to update it overall and be done
> with it.  Would you be willing to do this (preapproved; with
> UPDATE_COPYRIGHT_USE_INTERVALS unset)?
>
In light of what I said above, do you still believe the copyright bump
is really necessary?  If yes, I will amend the patch to include it.

Regards,
   Stefano