Avoid pruning files which were written after the prune process
starts, as it possible to concurrently create new objects while
"git prune" is running.
Tested on git.git by starting "git prune" in one terminal,
creating a random loose object via "git hash-object --stdin -w"
in a different terminal, and ensuring the loose object remains
after "git prune" completes.
Signed-off-by: Eric Wong <e@xxxxxxxxx>
---
I'm somewhat surprised this check didn't already exist;
but maybe nobody else runs prune manually, anymore.
builtin/prune.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/builtin/prune.c b/builtin/prune.c
index 8f4f052..d4cd054 100644
--- a/builtin/prune.c
+++ b/builtin/prune.c
@@ -14,6 +14,7 @@ static const char * const prune_usage[] = {
static int show_only;
static int verbose;
static unsigned long expire;
+static time_t start;
static int show_progress = -1;
static int prune_tmp_file(const char *fullpath)
@@ -21,7 +22,7 @@ static int prune_tmp_file(const char *fullpath)
struct stat st;
if (lstat(fullpath, &st))
return error("Could not stat '%s'", fullpath);
- if (st.st_mtime > expire)
+ if (st.st_mtime > expire || st.st_ctime >= start)
return 0;
if (show_only || verbose)
printf("Removing stale temporary file %s\n", fullpath);
@@ -47,7 +48,7 @@ static int prune_object(const unsigned char *sha1, const char *fullpath,
error("Could not stat '%s'", fullpath);
return 0;
}
- if (st.st_mtime > expire)
+ if (st.st_mtime > expire || st.st_ctime >= start)
return 0;
if (show_only || verbose) {
enum object_type type = sha1_object_info(sha1, NULL);
@@ -111,6 +112,7 @@ int cmd_prune(int argc, const char **argv, const char *prefix)
};
char *s;
+ start = time(NULL);
expire = ULONG_MAX;
save_commit_buffer = 0;
check_replace_refs = 0;
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html