On Sat, Apr 7, 2012 at 2:15 AM, Jim Meyering <j...@meyering.net> 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. >