summaryrefslogtreecommitdiff
path: root/builtin-pack-refs.c
diff options
context:
space:
mode:
authorJunio C Hamano <junkio@cox.net>2006-10-03 09:15:18 (GMT)
committerJunio C Hamano <junkio@cox.net>2006-10-03 09:15:18 (GMT)
commit03a182107fdb36170a72b8a3d94de2b52e3f6668 (patch)
treeecd3e4ff9cf3608b348bf845c0d626fd8ecf9b0c /builtin-pack-refs.c
parent47292d65dec754bbe37d82369faabeaf8f0cdb7a (diff)
downloadgit-03a182107fdb36170a72b8a3d94de2b52e3f6668.zip
git-03a182107fdb36170a72b8a3d94de2b52e3f6668.tar.gz
git-03a182107fdb36170a72b8a3d94de2b52e3f6668.tar.bz2
pack-refs: use lockfile as everybody else does.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-pack-refs.c')
-rw-r--r--builtin-pack-refs.c21
1 files changed, 4 insertions, 17 deletions
diff --git a/builtin-pack-refs.c b/builtin-pack-refs.c
index 4093973..ede4743 100644
--- a/builtin-pack-refs.c
+++ b/builtin-pack-refs.c
@@ -1,7 +1,6 @@
#include "cache.h"
#include "refs.h"
-static const char *result_path, *lock_path;
static const char builtin_pack_refs_usage[] =
"git-pack-refs [--prune]";
@@ -17,12 +16,6 @@ struct pack_refs_cb_data {
FILE *refs_file;
};
-static void remove_lock_file(void)
-{
- if (lock_path)
- unlink(lock_path);
-}
-
static int do_not_prune(int flags)
{
/* If it is already packed or if it is a symref,
@@ -69,6 +62,8 @@ static void prune_refs(struct ref_to_prune *r)
}
}
+static struct lock_file packed;
+
int cmd_pack_refs(int argc, const char **argv, const char *prefix)
{
int fd, i;
@@ -88,14 +83,7 @@ int cmd_pack_refs(int argc, const char **argv, const char *prefix)
if (i != argc)
usage(builtin_pack_refs_usage);
- result_path = xstrdup(git_path("packed-refs"));
- lock_path = xstrdup(mkpath("%s.lock", result_path));
-
- fd = open(lock_path, O_CREAT | O_EXCL | O_WRONLY, 0666);
- if (fd < 0)
- die("unable to create new ref-pack file (%s)", strerror(errno));
- atexit(remove_lock_file);
-
+ fd = hold_lock_file_for_update(&packed, git_path("packed-refs"), 1);
cbdata.refs_file = fdopen(fd, "w");
if (!cbdata.refs_file)
die("unable to create ref-pack file structure (%s)",
@@ -103,9 +91,8 @@ int cmd_pack_refs(int argc, const char **argv, const char *prefix)
for_each_ref(handle_one_ref, &cbdata);
fsync(fd);
fclose(cbdata.refs_file);
- if (rename(lock_path, result_path) < 0)
+ if (commit_lock_file(&packed) < 0)
die("unable to overwrite old ref-pack file (%s)", strerror(errno));
- lock_path = NULL;
if (cbdata.prune)
prune_refs(cbdata.ref_to_prune);
return 0;