Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

Original patch posted here: Amended patch to be applied posted here: On Thursday 02 December 2010, Stefano Lattarini wrote: > Now that Savannah is

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

Now that Savannah is up again, I'm pinging this patch again, according to this: I'll wait the customary 72 hours (until sunday evening) before pushing. Regards, Stefano

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

On Sunday 14 November 2010, Ralf Wildenhues wrote: > * Stefano Lattarini wrote on Thu, Nov 11, 2010 at 03:52:39PM CET: > > Pinging this patch again, following this: > > > > Patch is OK. I didn't check whether any of the t

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

* Stefano Lattarini wrote on Thu, Nov 11, 2010 at 03:52:39PM CET: > Pinging this patch again, following this: > Patch is OK. I didn't check whether any of the tests contained constructs problematic for set -e. > I'll wai

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

Pinging this patch again, following this: I'll wait the customary 72 hours (until sunday evening) before pushing. Regards, Stefano

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

Pinging this patch again, following this: I'll wait the customary 72 hours (until sunday evening) before pushing. Regards, Stefano

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

0 2001 From: Stefano Lattarini Date: Fri, 20 Aug 2010 19:53:38 +0200 Subject: [PATCH] Enable `set -e' in more tests (plus some tweakings). * tests/implicit.test: Enable `errexit' shell flag, and related changes. Add trailing `:' command. * tests/insh2.test: Likewise. * tests/instman2.

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

Tested with: 1. Debian GNU/Linux, GNU make 3.79, bash 2.0, gcc 2.95, autoconf/autom4te 2.67, in-tree build 2. Debian GNU/Linux, GNU make 3.81, bash 4.1.5, gcc 4.4.4, autoconf/autom4te 2.63, VAPTHin-tree build 3. Debian GNU/Linux, Korn shell 'Version JM 93t+ 2009-05-01', F

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

Tested with: 1. Debian GNU/Linux, GNU make 3.81, bash 4.1.5, gcc 4.4.4, autoconf/autom4te 2.67, in-tree build 2. Debian GNU/Linux, GNU make 3.80, Debian Alquist shell 0.5.5, gcc 4.4.4, autoconf/autom4te 2.65, VPATH build 3. Debian GNU/Linux, GNU make 3.79, bash 2.0, gcc 2.95,

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

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 exce

Re: [PATCH] Enable `set -e' in more tests (plus some tweakings).

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 >

[PATCH] Enable `set -e' in more tests (plus some tweakings).

roblems, as the modified tests do not use shell functions nor complex shell constructs, loops or conditionals. OK for maint? Regards, Stefano From 6029db4727931d0d49122b89c932ee0c24dc6542 Mon Sep 17 00:00:00 2001 From: Stefano Lattarini Date: Fri, 20 Aug 2010 22:33:44 +0200 Subject: [PATCH]

[PATCH] Enable `set -e' in more tests (plus some tweakings).

ditionals (with the exception of `info.test', but see the "FIXME" comment added to it). OK for maint? Regards, Stefano From 9dd345252eeb080ac8770a151fb1a0e9542383e7 Mon Sep 17 00:00:00 2001 From: Stefano Lattarini Date: Fri, 20 Aug 2010 19:53:38 +0200 Subject: [PATCH] Enabl