Re: [PATCH] [EVAL] with set -e exit the shell if a subshell exits non-zero

[Herbert Xu]

On Mon, Jun 14, 2010 at 09:56:03AM +0000, Gerrit Pape wrote:
> Example:
> 
> $ dash -c 'set -e; (false); echo here'
> here
> 
> With this commit, dash exits 1 before echo.
> 
> The bug was reported by Stefan Fritsch through
>  http://bugs.debian.org/514863
> 
> Signed-off-by: Gerrit Pape <p...@smarden.org>

Applied with slight change:

commit a92255d6fa8b8efb0f9fb093b77ac601fe3e9ede
Author: Gerrit Pape <p...@smarden.org>
Date:   Tue Jul 6 17:50:37 2010 +0800

    [EVAL] Check exit for eval NSUBSHELL
    
    Example:
    
    $ dash -c 'set -e; (false); echo here'
    here
    
    With this commit, dash exits 1 before echo.
    
    The bug was reported by Stefan Fritsch through
     http://bugs.debian.org/514863
    
    Signed-off-by: Gerrit Pape <p...@smarden.org>
    Signed-off-by: Herbert Xu <herb...@gondor.apana.org.au>

diff --git a/ChangeLog b/ChangeLog
index 3f5e24c..0e367a6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2010-07-06  Gerrit Pape <p...@smarden.org>
+
+       * Check exit for eval NSUBSHELL.
+
 2010-07-06  Herbert Xu <herb...@gondor.apana.org.au>
 
        * Fix loss of variables when hash collides.
diff --git a/src/eval.c b/src/eval.c
index 58739f5..5b8d36b 100644
--- a/src/eval.c
+++ b/src/eval.c
@@ -251,7 +251,7 @@ checkexit:
        case NSUBSHELL:
        case NBACKGND:
                evalfn = evalsubshell;
-               goto calleval;
+               goto checkexit;
        case NPIPE:
                evalfn = evalpipe;
                goto checkexit;
 
Thanks,
-- 
Email: Herbert Xu <herb...@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html