From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 12388 invoked by alias); 26 Sep 2017 18:22:36 -0000 Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm Precedence: bulk X-No-Archive: yes List-Id: Zsh Workers List List-Post: List-Help: X-Seq: 41768 Received: (qmail 20013 invoked by uid 1010); 26 Sep 2017 18:22:36 -0000 X-Qmail-Scanner-Diagnostics: from know-smtprelay-omc-1.server.virginmedia.net by f.primenet.com.au (envelope-from , uid 7791) with qmail-scanner-2.11 (clamdscan: 0.99.2/21882. spamassassin: 3.4.1. Clear:RC:0(80.0.253.65):SA:0(-4.7/5.0):. Processed in 2.476232 secs); 26 Sep 2017 18:22:36 -0000 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-4.7 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,SPF_PASS,T_DKIM_INVALID autolearn=ham autolearn_force=no version=3.4.1 X-Envelope-From: p.w.stephenson@ntlworld.com X-Qmail-Scanner-Mime-Attachments: | X-Qmail-Scanner-Zip-Files: | X-Originating-IP: [86.21.219.59] X-Authenticated-User: p.w.stephenson@ntlworld.com X-Spam: 0 X-Authority: v=2.1 cv=NqQsCJpJ c=1 sm=1 tr=0 a=utowdAHh8RITBM/6U1BPxA==:117 a=utowdAHh8RITBM/6U1BPxA==:17 a=L9H7d07YOLsA:10 a=9cW_t1CCXrUA:10 a=s5jvgZ67dGcA:10 a=kj9zAlcOel0A:10 a=x7bEGLp0ZPQA:10 a=P5Q6L1DsAAAA:8 a=mycQ_9az3xcSRkk31w8A:9 a=CjuIK1q_8ugA:10 a=jxi25fIUQso1YTL4nQQM:22 Date: Tue, 26 Sep 2017 19:15:26 +0100 From: Peter Stephenson To: Zsh hackers list Subject: Re: exit status 0 after SIGINT Message-ID: <20170926191526.1dbbf86f@ntlworld.com> In-Reply-To: <757d2c4c-fb9f-d168-c70a-08560677a181@inlv.org> References: <2f98bd49-1439-2bd9-c7b2-7d05ac8678fc@inlv.org> <20170925145717.13849173@pwslap01u.europe.root.pri> <757d2c4c-fb9f-d168-c70a-08560677a181@inlv.org> X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.28; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ntlworld.com; s=meg.feb2017; t=1506449728; bh=aHDV22tdAbnQpyu/m2za9FSKGP1l4JvhCOvFiXakIGc=; h=Date:From:To:Subject:In-Reply-To:References; b=Sdv/YacJ5UIiQWCvPEP8FoB56a0fMFPilmoadQtMjAI+7CqR7JPdMBbm1cvzLqgLS TDBtP9w1EDZz2ol7rc47XyarG2FpH1HcDlDCdUUwj0ibWUa3sIcvHUXbKOqWpEEltj sdNNL4xSTLUYrV7amQ8ke0SBVTlwSaASxb2tmNRzHR1c6tCI0cYNrHexZC3TKE+wCd coyhD/b7QWGWcLfKjQPuxgIxFaz8w3lAARMcvut+04y31AT54RWRE0mLRro1XYaPLI ouHoecwbInYUke/4kzzBLgWWwDE/QSP+lIcPcf1r06+YZYRSufeJCQZl2h5TMxdcDB sDZNiJGp78PjA== On Tue, 26 Sep 2017 11:57:33 +0100 Martijn Dekker wrote: > % kill -s INT $$ || echo oops > % echo $? > 1 > % kill -s INT $$ && echo oops > % echo $? > 130 > > Both should be 130. It's probably something like this. Hard to know under exactly what circumstance to change the value but leaving it alone if it's already non-zero doesn't seem too idiotic. Soureforge is down at the moment. pws diff --git a/Src/exec.c b/Src/exec.c index bd242d1..75a9f2b 100644 --- a/Src/exec.c +++ b/Src/exec.c @@ -3199,7 +3199,8 @@ execcmd_exec(Estate state, Execcmd_params eparams, } if (errflag) { - lastval = 1; + if (!lastval) + lastval = 1; if (oautocont >= 0) opts[AUTOCONTINUE] = oautocont; return;