From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=5.0 tests=MAILING_LIST_MULTI, RCVD_IN_DNSWL_NONE autolearn=ham autolearn_force=no version=3.4.4 Received: (qmail 32714 invoked from network); 27 Jun 2020 01:48:15 -0000 Received: from ns1.primenet.com.au (HELO primenet.com.au) (203.24.36.2) by inbox.vuxu.org with ESMTPUTF8; 27 Jun 2020 01:48:15 -0000 Received: (qmail 5026 invoked by alias); 27 Jun 2020 01:48:02 -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: Sender: zsh-workers@zsh.org X-Seq: 46134 Received: (qmail 27968 invoked by uid 1010); 27 Jun 2020 01:48:02 -0000 X-Qmail-Scanner-Diagnostics: from out2-smtp.messagingengine.com by f.primenet.com.au (envelope-from , uid 7791) with qmail-scanner-2.11 (clamdscan: 0.102.3/25850. spamassassin: 3.4.4. Clear:RC:0(66.111.4.26):SA:0(-2.6/5.0):. Processed in 4.959785 secs); 27 Jun 2020 01:48:02 -0000 X-Envelope-From: d.s@daniel.shahaf.name X-Qmail-Scanner-Mime-Attachments: | X-Qmail-Scanner-Zip-Files: | Received-SPF: none (ns1.primenet.com.au: domain at daniel.shahaf.name does not designate permitted sender hosts) X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduhedrudelvddghedvucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfgjfhfogggtgfesthejtddtredtvdenucfhrhhomhepffgrnhhi vghlucfuhhgrhhgrfhcuoegurdhssegurghnihgvlhdrshhhrghhrghfrdhnrghmvgeqne cuggftrfgrthhtvghrnhepfeduteegveehvdfhteevtedvudeludeijeeuudeujeejtdet uedtjefhtdfgieetnecukfhppeejledrudejiedrfeelrdeileenucevlhhushhtvghruf hiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpegurdhssegurghnihgvlhdrshhh rghhrghfrdhnrghmvg X-ME-Proxy: Date: Sat, 27 Jun 2020 01:47:17 +0000 From: Daniel Shahaf To: Sebastian Gniazdowski Cc: Zsh hackers list Subject: Re: [BUG] zsystem:34: flock: invalid timeout value: '0' Message-ID: <20200627014717.68986199@tarpaulin.shahaf.local2> In-Reply-To: References: <20200626141644.7cb5e511@tarpaulin.shahaf.local2> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sebastian Gniazdowski wrote on Fri, 26 Jun 2020 21:42 +0200: > On Fri, 26 Jun 2020 at 16:16, Daniel Shahaf wrote: > > Sebastian Gniazdowski wrote on Thu, 25 Jun 2020 19:49 +0200: > > > Hi > > > I think that something happened to zsystem / flock, as the following > > > returns the above error under Cygwin: > > > > > > $ zsystem flock -t 0 -f FD -e /home/SG/.zsh_nr1 > > > zsystem:34: flock: invalid timeout value: '0' > > > > > > Timeout of value 0 is an acceptable one - it means no waiting for the > > > lock, > > > just try-to-lock and exit. > > > > You've neglected to identify the environment it works in > > > What do you mean? I don't have access to environment other than Cygwin Your readers don't know this. > and I've written about the Cygwin-environment to narrow this report. Yes, but you didn't state the relevant details (version numbers of zsh, Cygwin, and Windows) of either the working case or the non-working case, nor the OS name of the non-working case. > > and the reason you didn't do > > a bisect as courtesy prescribes. > > > > In this case, system.c gets so few commits that a bisect is almost > > a formality. The bug is likely to have been introduced by workers/45708. > > > > I saw the subsecond timeout changes conversation/patch and assumed that > it's the cause, thus I've left the interested party to be alerted by the > title. It's not clear to me what made you assume that patch was the cause. At any rate, what you should have done is triage your own bug report and state your findings. What you had in fact done is posted a bug report and left it for others to triage. The part in the middle where you say "I have triaged this and I think this is caused by 45708" makes all the difference. If you don't say it, it won't have happened as far as your readers are concerned. > > I recommend that you test the behaviour with and without that commit > > and reported the results. > > > > I further recommend that, if you have time, you also prepare a patch, > > or at least an expected-to-fail regression test (using the 'f' flag > > after the exit code). > > > > I might prepare a patch, but I also think that the involved party might do > this. +1 I'm not going to comment any further about the process aspects. Regarding the actual bug report, the next steps are for the aforementioned commit to be confirmed as causing an unintended behaviour change and for a test and a patch to be written. Daniel