From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18379 invoked by alias); 26 Oct 2014 21:22:55 -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: 33546 Received: (qmail 25810 invoked from network); 26 Oct 2014 21:22:38 -0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.2 X-Originating-IP: [80.3.229.105] X-Spam: 0 X-Authority: v=2.1 cv=N7qnFgNB c=1 sm=1 tr=0 a=uz1KDxDNIq33yePw376BBA==:117 a=uz1KDxDNIq33yePw376BBA==:17 a=NLZqzBF-AAAA:8 a=kj9zAlcOel0A:10 a=q2GGsy2AAAAA:8 a=rSBAhrjrgKu-WzDqDV8A:9 a=CjuIK1q_8ugA:10 Date: Sun, 26 Oct 2014 21:22:34 +0000 From: Peter Stephenson To: zsh-workers@zsh.org Subject: Re: bug in zsh wait builtin - rhbz#1150541 Message-ID: <20141026212234.41fbb759@pws-pc.ntlworld.com> In-Reply-To: <141026134100.ZM14113@torch.brasslantern.com> References: <20141021210234.199eee3d@pws-pc.ntlworld.com> <141021235542.ZM14840@torch.brasslantern.com> <20141023093232.1f4201e2@pwslap01u.europe.root.pri> <141023215041.ZM19768@torch.brasslantern.com> <20141025200847.782316a4@pws-pc.ntlworld.com> <141025152827.ZM7060@torch.brasslantern.com> <141025153231.ZM19285@torch.brasslantern.com> <20141026000448.12fa03fd@pws-pc.ntlworld.com> <20141026001717.4d4e3f5a@pws-pc.ntlworld.com> <20141026190108.639ecd1c@pws-pc.ntlworld.com> <141026134100.ZM14113@torch.brasslantern.com> X-Mailer: Claws Mail 3.8.0 (GTK+ 2.24.7; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Sun, 26 Oct 2014 13:41:00 -0700 Bart Schaefer wrote: > On Oct 26, 7:01pm, Peter Stephenson wrote: > } Subject: Re: bug in zsh wait builtin - rhbz#1150541 > } > } Here's a test. > } > } +# once. The monitor option is irrelevant to the logic, so we'll make > } +# our job easier by turning it off. > } + unsetopt monitor > > That throws errors in some automated build environments. Look for other > places in A05 that fiddle with it. Looks like we need this. diff --git a/Test/A05execution.ztst b/Test/A05execution.ztst index 589815f..042b2d0 100644 --- a/Test/A05execution.ztst +++ b/Test/A05execution.ztst @@ -254,7 +254,7 @@ F:anonymous function, and a descriptor leak when backgrounding a pipeline # other words, we rely on the test working repeatedly rather than just # once. The monitor option is irrelevant to the logic, so we'll make # our job easier by turning it off. - unsetopt monitor + { unsetopt MONITOR } 2>/dev/null (exit 1) & one=$! (exit 2) & pws