Re: [PATCH v2 2/3] remote: repack packed-refs once when deleting multiple refs

2014-05-23 Thread Junio C Hamano
Junio C Hamano writes: >> -if (delete_ref(refname, sha1, 0)) >> +if (delete_ref(refname, NULL, 0)) >> result |= error(_("Could not remove branch %s"), >> refname); By the way, how does this series interact with what Ronnie and Michael are working on

Re: [PATCH v2 2/3] remote: repack packed-refs once when deleting multiple refs

2014-05-23 Thread Junio C Hamano
Jens Lindström writes: > When 'git remote rm' or 'git remote prune' were used in a repository > with many refs, and needed to delete many remote-tracking refs, a lot > of time was spent deleting those refs since for each deleted ref, > repack_without_refs() was called to rewrite packed-refs witho

[PATCH v2 2/3] remote: repack packed-refs once when deleting multiple refs

2014-05-23 Thread Jens Lindström
When 'git remote rm' or 'git remote prune' were used in a repository with many refs, and needed to delete many remote-tracking refs, a lot of time was spent deleting those refs since for each deleted ref, repack_without_refs() was called to rewrite packed-refs without just that deleted ref. To avo