[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, 3 May 2012 20:50:15 -0400 |
Any word on this?
On Thu, Apr 12, 2012 at 10:43 PM, Rocky Bernstein <address@hidden> wrote:
> 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.
>>
>
>
- bug#10317: PING - bug#10317: patch to su: -l and -p should not be used together,
Rocky Bernstein <=
bug#10317: PING - bug#10317: patch to su: -l and -p should not be used together, Jim Meyering, 2012/05/04