zsh-workers
 help / color / mirror / code / Atom feed
From: Axel Beckert <abe@deuxchevaux.org>
To: zsh-workers@zsh.org
Subject: Re: zpty-related testsuite failures if building in a chroot on a host running systemd 220 as init system
Date: Mon, 15 Jun 2015 12:55:11 +0200	[thread overview]
Message-ID: <20150615105510.GE9963@sym.noone.org> (raw)
In-Reply-To: <150613104333.ZM1660@torch.brasslantern.com> <20150613180154.7dbdd1a1@ntlworld.com>

Hi,

On Sat, Jun 13, 2015 at 06:01:54PM +0100, Peter Stephenson wrote:
> On Sat, 13 Jun 2015 12:48:58 +0200
> Axel Beckert <abe@deuxchevaux.org> wrote:
> > The only thing which is IMHO left on the zsh side is why the testsuite
> > fails (instead of e.g. skipping the according tests) if /dev/pts/ptmx
> > is missing. Is this on purpose? Or just never occurred before? (Mostly
> > curious. :-)
> 
> Sounds like you've got to the bottom of this, which is good.

Yes, I'm glad, too. Especially because afterwards I could upload zsh
5.0.8 to Debian Unstable with good conscience. :-)

It so far build fine on all architectures except GNU Hurd:
https://buildd.debian.org/status/package.php?p=zsh

5.0.8 on sparc hasn't been built yet and the build failure on GNU Hurd
seems to have been an endless loop in the configure script which means
that the issue is rather in autotools than in zsh.

On Sat, Jun 13, 2015 at 10:43:33AM -0700, Bart Schaefer wrote:
> On Jun 13,  6:01pm, Peter Stephenson wrote:
> }
> } I guess it would be possible to have a "broken posxix_openpt" test ---
> 
> Not at configure time -- that would result in the module being left
> uncompiled, when in fact outside of the chroot it would work.

Yeah, I'd say the test suite should not expect to be run in the same
environment as the build. E.g. in Debian we run it directly after the
build inside the build environment, but also "as installed"
afterwards: http://ci.debian.net/packages/z/zsh/unstable/amd64/ (5.0.8
is still missing there, as there may occur lags up to a few days.)

> I think the test failing here was exactly the right thing, as it led to
> the origin of the problem being discovered.

Yes and no. I'm glad about it failing, yes.

> The only viable alternative would be to report the test as "skipped"

That's what I imagined.

> which I guess would be OK for the secondary tests that rely on zpty,
> but not for V08zpty itself.

Hrm. If I know, I the zpty tests would fail due to the environment,
I'd tend to skip them as I would skip 64-bit-only tests in a 32-bit
environment.

> I suppose the test suite could remember that V08 failed and therefore
> skip X02 and all of Y, so as to isolate the problem better.  We could
> add some sort of dependency check.

Actually I think that's overkill. It was clear for me in this case
that all failing tests were zpty related.

		Kind regards, Axel
-- 
/~\  Plain Text Ribbon Campaign                   | Axel Beckert
\ /  Say No to HTML in E-Mail and News            | abe@deuxchevaux.org  (Mail)
 X   See http://www.nonhtmlmail.org/campaign.html | abe@noone.org (Mail+Jabber)
/ \  I love long mails: http://email.is-not-s.ms/ | http://abe.noone.org/ (Web)


  parent reply	other threads:[~2015-06-15 10:55 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-05 15:02 Axel Beckert
2015-06-05 15:29 ` Axel Beckert
2015-06-06 10:49 ` Oliver Kiddle
2015-06-06 18:48   ` Axel Beckert
2015-06-12 20:52     ` Axel Beckert
2015-06-13 10:48       ` Axel Beckert
2015-06-13 17:01         ` Peter Stephenson
2015-06-13 17:43           ` Bart Schaefer
2015-06-15 10:55           ` Axel Beckert [this message]
2015-06-16  0:07             ` Bart Schaefer

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=20150615105510.GE9963@sym.noone.org \
    --to=abe@deuxchevaux.org \
    --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).