Re: [PATCH v2 09/17] Add watchman support to reduce index refresh cost
- From
David Turner <dturner@twopensource.com>
- Date
- Mar 24, 2016, 17:58 UTC
- Message-ID
- <1458842335.28595.10.camel@twopensource.com>
- In-Reply-To
- <56F3EFF4.30500@jeffhostetler.com>
On Thu, 2016-03-24 at 09:47 -0400, Jeff Hostetler wrote:
Show 8 quoted lines
> I'm seeing wm->name have value ".git" rather than ".git/" on Linux. > > > On 03/18/2016 09:04 PM, David Turner wrote: > > + if (!strncmp(wm->name, ".git/", 5) || > > + strstr(wm->name, "/.git/")) > > + continue; >
Thanks. I don't think I considered the case of .git itself being modified, although clearly files are occasionally created/deleted in there. I'm going to fix this by ignoring all directories, since we'll capture untracked-cache changes via files in those dirs.