[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: test clean-up: use env kill, not "$abs_top_builddir/src/kill"
From: |
Pádraig Brady |
Subject: |
Re: test clean-up: use env kill, not "$abs_top_builddir/src/kill" |
Date: |
Wed, 02 Jul 2014 10:59:42 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
On 07/01/2014 06:32 PM, Jim Meyering wrote:
> On Tue, Jul 1, 2014 at 9:58 AM, Pádraig Brady <address@hidden> wrote:
>> On 07/01/2014 05:49 PM, Jim Meyering wrote:
>>> I wrote "$abs_top_builddir/src/kill" to ensure that we'd run the
>>> just-build coreutils binary, and not a shell built-in function by
>>> the same name -- but that was back before I discovered
>>> that "env builtin" is a much better way to do the job.
>>>
>>> I changed some of these while fixing the pwd uses.
>>> This fixes the kill-related ones:
>>>
>> There is also a "$abs.../src/test" one at least that
>> could be merged into this I think
>
> Yeah, I saw those two, but confess I was lazy -- it required
> generalizing the commit message. Since you called me on
> it, I've done it. Thanks :-) Also fixed a typo s/rathe/rather/
> in the commit log:
>
Looks good.
thanks!