From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14380 invoked by alias); 14 Mar 2018 11:04:57 -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: List-Unsubscribe: X-Seq: 42461 Received: (qmail 5653 invoked by uid 1010); 14 Mar 2018 11:04:57 -0000 X-Qmail-Scanner-Diagnostics: from mailout2.w1.samsung.com 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(210.118.77.12):SA:0(-6.9/5.0):. Processed in 3.898445 secs); 14 Mar 2018 11:04:57 -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=-6.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_HI, RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_PASS,SPF_PASS,T_DKIM_INVALID, T_RP_MATCHES_RCVD autolearn=ham autolearn_force=no version=3.4.1 X-Envelope-From: p.stephenson@samsung.com X-Qmail-Scanner-Mime-Attachments: | X-Qmail-Scanner-Zip-Files: | DKIM-Filter: OpenDKIM Filter v2.11.0 mailout2.w1.samsung.com 20180314105445euoutp02a04397a7793f3e808a58606f95508515~bxBMqhHAF2580225802euoutp02j DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1521024885; bh=wrK0D8qWmI7ilJFDT/UleTEKC6mnumvhqKDRRAco+yY=; h=Date:From:To:Subject:In-reply-to:References:From; b=r0M4Sadsm1RlmEjsZ5+BmZMZiNcrxpfFgoMUtudV8IdK8B7thojauNJeB3whEcI5b ECOSMXF321Yj/zaWYwp7mkRQTA+cxEVxb12HgsRFDhCoWpskkCc+FgdlCjnHmIjLBF eK1ilNuB/EByr92gWrk6cVW2Y8583stLXeRQqsjY= X-AuditID: cbfec7f5-b5fff700000028a9-75-5aa8ff74e7ff Date: Wed, 14 Mar 2018 10:54:42 +0000 From: Peter Stephenson To: Zsh hackers list Subject: Re: $( In-reply-to: <20180314103254.GA10404@chaz.gmail.com> Organization: SCSC X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.30; x86_64-pc-linux-gnu) MIME-version: 1.0 Content-type: text/plain; charset="US-ASCII" Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrKIsWRmVeSWpSXmKPExsWy7djPc7ol/1dEGbzfo2NxsPkhkwOjx6qD H5gCGKO4bFJSczLLUov07RK4MnYefc5cMIO9YvWFDtYGxhusXYycHBICJhLbll5lBrGFBFYw Suy/K9bFyAVk9zJJfG+4DVfU82UtG0RiGaPE633L2CGcaUwSPauPsEA4Zxglrq++zAjhHGSU uNdyEqyfRUBVYt3zfUwgNpuAocTUTbMZQWwRAS2JHSdPgsWFBYwkVnetZgGxeYH2ff2xBKyG E8hedGcnUA0HB7+AkMSFZluIk+wlju6BaOUVEJT4MfkeWCuzgI7Etm2P2SFseYnNa94yg9wj IdDDJvFu12JGiGYXiSM7zzBD2MISr45vYYewZSQuT+5mgWhoZpRYe/8+G0Sih1Fi1uJQCNta ou/2RUaIDXwSk7ZNZwY5TkKAV6KjTQiixEPi5+7jUPMdJTr7DjBDAqWdUeLs5rVsExjlZyE5 fBaSw2chOXwBI/MqRvHU0uLc9NRi47zUcr3ixNzi0rx0veT83E2MwMg//e/41x2M+/4kHWIU 4GBU4uFluLA8Sog1say4MvcQowQHs5II71aZFVFCvCmJlVWpRfnxRaU5qcWHGKU5WJTEeeM0 6qKEBNITS1KzU1MLUotgskwcnFINjLlym1dFCavO/hVik6/zRc5mPaOkzJGmG97ubR5lNobL RCSalC9Y9V13WKY66duEqBs1lj1v+C1yRJ2O6bLK2GY0/it/l5Tz+KODR8Avbb3sx9q+LX9Y Nf5M3M3v6uX1ZfvifSUr2b1fhwcyX7Zv25rTve3/3BaHbVwuU1NOzrK7a1P2XuCmEktxRqKh FnNRcSIAwRe9gPgCAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrILMWRmVeSWpSXmKPExsVy+t/xy7ol/1dEGTx5x29xsPkhkwOjx6qD H5gCGKO4bFJSczLLUov07RK4MnYefc5cMIO9YvWFDtYGxhusXYycHBICJhI9X9aygdhCAksY JaasL4GwZzBJ/F+SC2GfY5SYOEG6i5ELyD7IKNG2/hkTSIJFQFVi3fN9YDabgKHE1E2zGUFs EQEtiR0nT4LFhQWMJFZ3rWYBsXmBln39sQSshhPIXnRnJxPE0E5GiRm7DwBdwcHBLyAkcaHZ FuI4e4mjeyDm8AoISvyYfA9sDjPQ/M3bmlghbHmJzWveMkMcqi5x4+5u9gmMQrOQtMxC0jIL ScsCRuZVjCKppcW56bnFRnrFibnFpXnpesn5uZsYgQG77djPLTsYu94FH2IU4GBU4uFluLA8 Sog1say4MvcQowQHs5II71aZFVFCvCmJlVWpRfnxRaU5qcWHGKU5WJTEec8bVEYJCaQnlqRm p6YWpBbBZJk4OKUaGCXnXfrzRO5hTvxb9iNHza5tT6iPLM576K+bbnBAoqSpJGLjgVw10/N8 bjaKgu/u2vaFvrb7PmveE9MLe8N2aeVEiHlmHY5vPpggpLRr0qGfjV8+J13Ridl/LNTchynB eqPupD9TpMIWvJh3h4lr4wq+8xK7xPVv/46yPXfIYOKHg84tdxqKi5RYijMSDbWYi4oTAaAQ wpRUAgAA X-CMS-MailID: 20180314105444eucas1p1bf1f8d90c04d052844c2d5f87f40ebb0 X-Msg-Generator: CA CMS-TYPE: 201P X-CMS-RootMailID: 20180314103335epcas4p30222f0df02adda27cbddbe62075ff9ad X-RootMTR: 20180314103335epcas4p30222f0df02adda27cbddbe62075ff9ad References: <20180314103254.GA10404@chaz.gmail.com> On Wed, 14 Mar 2018 10:32:54 +0000 Stephane Chazelas wrote: > $ zsh -c 'ar=$( zsh:1: no such file or directory: file > 0 > > That's different from other shells (ksh93, bash, mksh) that > return 1, and is also inconsistent with the cases where $(<) is > not recognised as that operator and runs the NULLCMD instead: We set lastval = cmdoutval if it's an ordinary command substitution, so it's easy to make this consistent with that behaviour. Doing so seems unproblematic. pws diff --git a/Src/exec.c b/Src/exec.c index e5c6455..299b816 100644 --- a/Src/exec.c +++ b/Src/exec.c @@ -4514,6 +4514,7 @@ getoutput(char *cmd, int qt) untokenize(s); if ((stream = open(unmeta(s), O_RDONLY | O_NOCTTY)) == -1) { zwarn("%e: %s", errno, s); + lastval = cmdoutval = 1; return newlinklist(); } return readoutput(stream, qt);