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

[Rocky Bernstein]

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.
>>
>
>