diff options
| author | Michael Haggerty <mhagger@alum.mit.edu> | 2013-06-20 10:37:46 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2013-06-20 15:50:17 -0700 |
| commit | 9f69d297703bff37c5506276c2565c721347e03f (patch) | |
| tree | e2a24bfaae82ea15b70fa15a443594483bccedae /builtin/clone.c | |
| parent | 2fff7812902abd0afe05ae1e9ef334fcd26f0389 (diff) | |
| download | git-9f69d297703bff37c5506276c2565c721347e03f.tar.gz git-9f69d297703bff37c5506276c2565c721347e03f.tar.xz | |
refs: implement simple transactions for the packed-refs file
Handle simple transactions for the packed-refs file at the
packed_ref_cache level via new functions lock_packed_refs(),
commit_packed_refs(), and rollback_packed_refs().
Only allow the packed ref cache to be modified (via add_packed_ref())
while the packed refs file is locked.
Change clone to add the new references within a transaction.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/clone.c')
| -rw-r--r-- | builtin/clone.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/builtin/clone.c b/builtin/clone.c index 66bff5700..14b132356 100644 --- a/builtin/clone.c +++ b/builtin/clone.c @@ -493,13 +493,16 @@ static void write_remote_refs(const struct ref *local_refs) { const struct ref *r; + lock_packed_refs(LOCK_DIE_ON_ERROR); + for (r = local_refs; r; r = r->next) { if (!r->peer_ref) continue; add_packed_ref(r->peer_ref->name, r->old_sha1); } - pack_refs(PACK_REFS_ALL); + if (commit_packed_refs()) + die_errno("unable to overwrite old ref-pack file"); } static void write_followtags(const struct ref *refs, const char *msg) |