Fwd: [PATCH] Invoke "git gc --auto" from "git add" and "git fetch"
From: Govind Salinas <hidden>
Date: 2016-06-15 22:43:33
Forgot to cc the list. ---------- Forwarded message ---------- From: Govind Salinas <redacted> Date: Sep 5, 2007 3:57 PM Subject: Re: [PATCH] Invoke "git gc --auto" from "git add" and "git fetch" To: Junio C Hamano <redacted> I have a completely uninformed question... Can git-add/rm/etc create dangling object or objects that would be cleaned up by git-gc --auto? I would think (and I could be completely off base here) that you would only want to call gc after an operation that could create stuff that needs to be gc'ed, since only then could the threshold be reached. Anyways, just curious. One day I should actually go in and read some git code. -Govind On 9/5/07, Junio C Hamano [off-list ref] wrote:
quoted hunk
This makes the two commands to call "git gc --auto" when they are done. I earlier said that obvious candidates also include merge and rebase, but these are lot less frequent operations compared to add, and more importantly, in a normal workflow they would almost always happen after "git fetch" is done. In other words, if you are downstream developer, the automatic invocation in "git fetch" will take care of things for you, and otherwise if you do not have an upstream, you would be doing your own development, so "git add" to add your changes will take care of the auto invocation for you. Signed-off-by: Junio C Hamano <redacted> --- * This is obviously a follow-up to the previous one that allows you to say "git gc --auto". I somewhat feel dirty about calling cmd_gc() bypassing fork & exec from "git add", though... builtin-add.c | 2 ++ git-fetch.sh | 1 + 2 files changed, 3 insertions(+), 0 deletions(-)diff --git a/builtin-add.c b/builtin-add.c index 105a9f0..8431c16 100644 --- a/builtin-add.c +++ b/builtin-add.c@@ -263,9 +263,11 @@ int cmd_add(int argc, const char **argv, const char *prefix) finish: if (active_cache_changed) { + const char *args[] = { "gc", "--auto", NULL }; if (write_cache(newfd, active_cache, active_nr) || close(newfd) || commit_locked_index(&lock_file)) die("Unable to write new index file"); + cmd_gc(2, args, NULL); } return 0;diff --git a/git-fetch.sh b/git-fetch.sh index c3a2001..86050eb 100755 --- a/git-fetch.sh +++ b/git-fetch.sh@@ -375,3 +375,4 @@ case "$orig_head" in fi ;; esac +git gc --auto --1.5.3.1.840.g0fedbc - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html