[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#10317: PING - bug#10317: patch to su: -l and -p should not be used t

From: Rocky Bernstein
Subject: bug#10317: PING - bug#10317: patch to su: -l and -p should not be used together
Date: Thu, 12 Apr 2012 22:43:44 -0400

On Sat, Apr 7, 2012 at 2:15 AM, Jim Meyering <address@hidden> wrote:

> Rocky Bernstein wrote:
> > Any progress or thoughts on the revised patch?
> Hi Rocky,
> Sorry about the delay.
> I haven't forgotten.
> I noticed that your test requires typing a root password.

Um, no. I don't know how you got that impression, but did you try running
the test as a non-root or non-privileged user?

Here is what I got when I just tried:

$ (cd tests && make check TESTS=su/p-and-l)
make  check-TESTS
make[1]: Entering directory `/src/external-vcs/coreutils/tests'
make[2]: Entering directory `/src/external-vcs/coreutils/tests'
PASS: su/p-and-l
1 test passed

(Sorry for the delay - I wrote this long enough ago that  I didn't remember
what I did and only now have I had the time to review.)

We prefer to avoid that, so that the tests can be run
> non-interactively.  That's why some tests are root-only.
> (see README for details on running them)
> They're marked with require_root_.
> At least one is a bourne shell husk that actually does
> its work in Perl, so may be a good model.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]