zsh-workers
 help / color / mirror / code / Atom feed
From: Bart Schaefer <schaefer@brasslantern.com>
To: zsh-workers@zsh.org
Subject: Re: zsh 5.0.5-dev-2 / Occassional hangs on Test/A05execution.ztst
Date: Fri, 15 Aug 2014 10:33:30 -0700	[thread overview]
Message-ID: <140815103330.ZM17468@torch.brasslantern.com> (raw)
In-Reply-To: <20140815093255.GN6176@sym.noone.org>

On Aug 15, 11:32am, Axel Beckert wrote:
} 
} > Actually the warning could be stronger than that, because the
} > inability to setpt MONITOR renders the test invalid
} 
} So instead of or in addition to the warning, can we skip this test in
} case of no controlling terminal?

The test that has the warning and the test that is breaking for you are
two different tests.  The test that's "invalid" is NOT the one that is
causing your issue, so skipping the test that has the warning would not
resolve this discussion thread.  More specifically:

} > (it will always succeed, even in the case we are trying to regress).
} 
} Well, it at least hangs under some yet to be determined conditions.

The "will always succeed" statement does NOT apply to the test that is
hanging; sorry that wasn't clear enough.  The test that is hanging in
fact does NOT WANT the monitor option and is *intended* to regress
something that happens with no controlling terminal!

Which means that maybe it still happens, though why "kill -INT" is not
sufficient to end the test may have something to do with the build
environment (e.g., perhaps the auto-build blocks the signal because it
doesn't want anything it's building to be able to stop the entire
automation).  

} (The one case where it seems to hang for me with a controlling
} terminal, I wonder if I was just not patient enough. I think I waited
} for at least 20 to 30 seconds before pressing Ctrl-C, but I'm no more
} sure.)

If you had to wait more than 5 seconds plus some rounding error, you
had waited long enough.

I've committed a change to add the 6-second timeout on the final read
at the end of the test, see if that changes anything ...


  reply	other threads:[~2014-08-15 17:33 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-12 20:29 zsh 5.0.5-dev-2 Peter Stephenson
2014-08-12 22:36 ` Axel Beckert
2014-08-13 14:04   ` zsh 5.0.5-dev-2 / Occassional hangs on Test/A05execution.ztst Axel Beckert
2014-08-13 19:08     ` Bart Schaefer
2014-08-14  8:48       ` Axel Beckert
2014-08-14  8:50       ` Peter Stephenson
2014-08-14 16:25         ` Bart Schaefer
2014-08-14 16:31           ` Axel Beckert
2014-08-14 17:34             ` Axel Beckert
2014-08-15  4:50               ` Bart Schaefer
2014-08-15  9:32                 ` Axel Beckert
2014-08-15 17:33                   ` Bart Schaefer [this message]
2014-08-15 19:42                     ` Axel Beckert
2014-08-16 18:12                       ` zsh 5.0.5/6 Peter Stephenson
2014-08-13 17:28   ` zsh 5.0.5-dev-2 Dominic Hopf
2014-08-13 22:34 ` Oliver Kiddle
2014-08-14  8:34   ` Peter Stephenson
2014-08-14  9:32     ` Peter Stephenson
2014-08-14 16:20       ` Bart Schaefer
2014-08-14 19:54         ` Peter Stephenson
2014-08-15  4:44           ` Bart Schaefer
2014-08-15  8:38             ` Peter Stephenson
2014-08-15 11:23             ` Han Pingtian
2014-08-15 17:17               ` Bart Schaefer
2014-08-16  0:35                 ` Han Pingtian
2014-08-17 17:30                   ` Bart Schaefer
2014-08-18  2:56                     ` Han Pingtian
2014-08-18  6:36                       ` Bart Schaefer
2014-08-18 14:02                         ` Han Pingtian
2014-08-23 17:51                     ` Symlink chasing 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=140815103330.ZM17468@torch.brasslantern.com \
    --to=schaefer@brasslantern.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).