zsh-workers
 help / color / mirror / code / Atom feed
From: Peter Stephenson <p.w.stephenson@ntlworld.com>
To: zsh-workers@zsh.org, "İsmail Dönmez" <ismail@i10z.com>
Subject: Re: W02jobs.ztst hangs with zsh 5.6.1
Date: Mon, 10 Sep 2018 12:19:49 +0100 (BST)	[thread overview]
Message-ID: <524677991.1156882.1536578389106@mail2.virginmedia.com> (raw)
In-Reply-To: <CAMAzfThnh_rVT_f9ccSvYoqCLhhT8z1sdm9vpBKP+TM+X5YmCw@mail.gmail.com>

> On 10 September 2018 at 11:39 İsmail Dönmez <ismail@i10z.com> wrote:
> Running test: `jobs -r` and `jobs -s` with suspended job
> ZTST_test: expecting status: 0
> Input: /tmp/zsh.ztst.13823/ztst.in, output:
> /tmp/zsh.ztst.13823/ztst.out, error: /tmp/zsh.ztst.13823/ztst.terr
> 
> (*.in, *.terr is empty), so:
> havana ~ > cat /tmp/zsh.ztst.13823/ztst.out
> zsh:*(stopped|suspended)*sleep*
> -
> \[1]  + (stopped|suspended)*sleep*

Thanks --- the test is doing the following.

sleep 5
^Z
# Should print stopped / suspended message.
jobs -r
# Shouldn't print anything.
print -- -
# Should print "-" --- not really important here, it's
# just to sync up the output within the test system.
jobs -s
# Should show the job.
# The test doesn't bother with the following, it just exits, but
# to try it out by hand I suggest...
kill %1

Could you try this by hand to check?  I'd be very surprised if
it failed, as all the hairiness here is in the use of zpty in the
test system, which I've had a lot of trouble with when trying to
add new tests.  Again, I suspect pre-existing races rather than
the explicit effect of new changes.

In that case I think we'll just have to disable much of W02.

Thanks!
pws

  reply	other threads:[~2018-09-10 11:20 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20180910102119epcas5p384cea9ba5749eb81be99c4f4d85a954e@epcas5p3.samsung.com>
2018-09-10 10:20 ` İsmail Dönmez
2018-09-10 10:32   ` Peter Stephenson
2018-09-10 10:39     ` İsmail Dönmez
2018-09-10 11:19       ` Peter Stephenson [this message]
2018-09-10 11:32         ` İsmail Dönmez
2018-09-10 12:46           ` Peter Stephenson
2018-09-10 13:07             ` İsmail Dönmez
2018-09-10 15:14         ` Peter Stephenson
2018-09-10 15:50           ` Daniel Shahaf
2018-09-10 16:22             ` Peter Stephenson
2018-09-10 16:55               ` Bart Schaefer
2018-09-10 17:40                 ` Peter Stephenson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=524677991.1156882.1536578389106@mail2.virginmedia.com \
    --to=p.w.stephenson@ntlworld.com \
    --cc=ismail@i10z.com \
    --cc=zsh-workers@zsh.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).