GNU bug report logs - #8288
vc-git slow on large repository

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 8288 in the body.
You can then email your comments to 8288 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: S Boucher <stbya <at> yahoo.com>
To: bug-gnu-emacs <at> gnu.org
Subject: vc-git slow on large repository
Date: Fri, 18 Mar 2011 12:32:17 -0700 (PDT)

find-file and save-buffer are slow.  It appears to be due to vc-git-state.

vc-git-state calls:

     git diff-index -z HEAD -- file.cpp

Which is slow on large git repository, such as webkit.

The question is then whether or not vc-git.el can be more clever than it is now.

I don't have the answer to this.


System:
ubuntu 10.04 64bit
emacs 23.2
git 1.7.0.4
8G of ram
Intel Core i7 (8 processors)

Message #8 received at 8288 <at> debbugs.gnu.org (full text, mbox):

From: S Boucher <stbya <at> yahoo.com>
To: 8288 <at> debbugs.gnu.org
Subject: Re: bug#8288: Acknowledgement (vc-git slow on large repository)
Date: Fri, 18 Mar 2011 19:54:38 -0700 (PDT)

I think 'git diff-index -z HEAD -- <file>' could be replaced with 'git status -z 
--porcelain <file>' in vc-git-state.

It is 10times faster on a large webkit repository.




----- Original Message ----
> From: GNU bug Tracking System <help-debbugs <at> gnu.org>
> To: S Boucher <stbya <at> yahoo.com>
> Sent: Fri, March 18, 2011 3:33:02 PM
> Subject: bug#8288: Acknowledgement (vc-git slow on large repository)
> 
> Thank you for filing a new bug report with GNU.
> 
> This is an automatically  generated reply to let you know your message
> has been received.
> 
> Your  message is being forwarded to the package maintainers and other
> interested  parties for their attention; they will reply in due course.
> 
> Your message  has been sent to the package maintainer(s):
> bug-gnu-emacs <at> gnu.org
> 
> If you wish  to submit further information on this problem, please
> send it to 8288 <at> debbugs.gnu.org.
> 
> Please do  not send mail to help-debbugs <at> gnu.org unless you  wish
> to report a problem with the Bug-tracking system.
> 
> -- 
> 8288: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=8288
> GNU Bug Tracking  System
> Contact help-debbugs <at> gnu.org with problems
> 

Message #11 received at 8288 <at> debbugs.gnu.org (full text, mbox):

From: Andreas Schwab <schwab <at> linux-m68k.org>
To: S Boucher <stbya <at> yahoo.com>
Cc: 8288 <at> debbugs.gnu.org
Subject: Re: bug#8288: Acknowledgement (vc-git slow on large repository)
Date: Sat, 19 Mar 2011 08:32:02 +0100

S Boucher <stbya <at> yahoo.com> writes:

> I think 'git diff-index -z HEAD -- <file>' could be replaced with 'git status -z 
> --porcelain <file>' in vc-git-state.

This requires at least git 1.7.0, but it can be replaced by --short in
older versions.

Andreas.

-- 
Andreas Schwab, schwab <at> linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Message #14 received at 8288 <at> debbugs.gnu.org (full text, mbox):

From: Jason Merrill <jason <at> redhat.com>
To: 8288 <at> debbugs.gnu.org
Subject: vc-git slow on large repository
Date: Tue, 14 Jun 2011 19:40:40 -0400

This issue has been very frustrating for me with GCC git; when the 
repository isn't in the cache saving a file can cause emacs to freeze 
for about 10 seconds.  I work around it with

(defun vc-git-state-heuristic (file)
  "Just claim we're up to date."
  'up-to-date)

Message #17 received at 8288 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Andreas Schwab <schwab <at> linux-m68k.org>
Cc: S Boucher <stbya <at> yahoo.com>, 8288 <at> debbugs.gnu.org
Subject: Re: bug#8288: Acknowledgement (vc-git slow on large repository)
Date: Sun, 30 Jun 2019 06:06:23 +0200

tags 8288 fixed
close 8288 26.1
quit

Andreas Schwab <schwab <at> linux-m68k.org> writes:
>> I think 'git diff-index -z HEAD -- <file>' could be replaced with 'git status -z
>> --porcelain <file>' in vc-git-state.
>
> This requires at least git 1.7.0, but it can be replaced by --short in
> older versions.

vc-git-state now uses the alternative faster method.

This change was made in:

24301c81 Speed up vc-git-status and make it more precise
Mon, 1 May 2017 03:30:01 +0200 (04:30 +0300)
https://git.savannah.gnu.org/gitweb/?p=emacs.git;a=commit;h=24301c8148f5f3220d7e597c73a59551cfa10eea

I'm therefore closing this bug report.  Please re-open if you
disagree.

Thanks,
Stefan Kangas

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.