public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: libc-alpha@sourceware.org
Subject: Re: [PATCH] tst-ttyname: skip the test when /dev/ptmx is not available
Date: Fri, 29 Dec 2017 14:49:00 -0000	[thread overview]
Message-ID: <20171229144945.GA5698@altlinux.org> (raw)
In-Reply-To: <87efndljui.fsf@mid.deneb.enyo.de>

[-- Attachment #1: Type: text/plain, Size: 1010 bytes --]

On Fri, Dec 29, 2017 at 02:55:17PM +0100, Florian Weimer wrote:
> * Dmitry V. Levin:
> 
> >> I don't think the glibc test suite is supposed to pass in such an
> >> environment.
> >
> > It used to work perfectly for at least 15 years, and it still works
> > when tst-ttyname is fixed.
> 
> This just reflects that we did not have sufficient test coverage for
> the PTY code.

No doubts about that, but the kernel side of pty interface does not have
to be implemented, so we cannot allow the test to fail just because the
kernel does not provide it.

> > btw, as support_become_root is also unavailable in restricted
> > environments, an alternative fix for tst-ttyname is to skip the test
> > when support_become_root fails.
> 
> But that's unrelated.  And in some environments, support_become_root
> can fail, but the process was initially sufficiently privileged to run
> the test.

I don't think such environments exist in practice but this is
theoretically possible.


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

  reply	other threads:[~2017-12-29 14:49 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-25 11:48 Dmitry V. Levin
2017-12-25 18:19 ` Florian Weimer
2017-12-25 19:13   ` Dmitry V. Levin
2017-12-25 20:10     ` Zack Weinberg
2017-12-25 21:01       ` Dmitry V. Levin
2017-12-25 21:19         ` Zack Weinberg
2017-12-25 21:39           ` Dmitry V. Levin
2017-12-26 13:07         ` Florian Weimer
2017-12-26 13:53           ` Dmitry V. Levin
2017-12-26 23:07             ` Dmitry V. Levin
2018-01-01  1:30               ` Joseph Myers
2018-01-01  2:28                 ` Luke Shumaker
2017-12-29 14:20             ` Florian Weimer
2017-12-29 14:49               ` Dmitry V. Levin [this message]
2018-01-02  5:19               ` [PATCH v3] tst-ttyname: skip the test if failed to become root Luke Shumaker
2017-12-30 23:09           ` Luke Shumaker
2017-12-31  0:11             ` Dmitry V. Levin
2017-12-26 14:10 Dmitry V. Levin

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=20171229144945.GA5698@altlinux.org \
    --to=ldv@altlinux.org \
    --cc=libc-alpha@sourceware.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.
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 read-only IMAP folder(s) and NNTP newsgroup(s).