From: Duy Nguyen <pclouds@gmail.com>
Date: Tue, 10 May 2016 11:31:18 GMT
Subject: Re: [PATCH v9 03/19] index-helper: new daemon for caching index and related stuff
Message-ID: <CACsJy8BE5jjr=TQVyRNz0M2yxTsoXNQ=fU80rGDcm-SXy4ktcA@mail.gmail.com>
In-Reply-To: <20160510131228.Horde.WHWUaWE4DeqFLnoMEbjxcxG@webmail.informatik.kit.edu>

On Tue, May 10, 2016 at 6:12 PM, SZEDER Gábor <szeder@ira.uka.de> wrote:
>
> Quoting Duy Nguyen <pclouds@gmail.com>:
>
>> This reminds me, we need to update command-list.txt as well. I guess
>> it belongs to purehelpers? Back to the patch..
>>
>> On Tue, May 10, 2016 at 5:13 PM, SZEDER Gábor <szeder@ira.uka.de> wrote:
>>>
>>> This patch adds a new plumbing command, which then will show up in
>>> completion after 'git <TAB>'.  Could you please squash in this
>>> oneliner to exclude index-helper from porcelain commands in the
>>> completion script?
>>
>>
>> I don't think the user should run this command explicitly a lot.
>> Usually it should be run automatically (with config key
>> indexhelper.autorun) and die when idle for some time. You just don't
>> care about it (until performance degrades and you have to
>> investigate). Occasionally you may have to "git index-helper --kill".
>> Is there any other reason it should be run often enough to justify
>> this change?
>
>
> I think you misunderstood this change, it _excludes_
> 'index-helper' from the list of porcelain commands.

Oops, right it excludes it.
-- 
Duy