diff options
author | Ronnie Sahlberg <sahlberg@google.com> | 2014-06-20 18:42:48 +0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-07-14 22:54:41 +0400 |
commit | 447ff1bf0acf9a1d7d2dc3aed032c209f105fb8a (patch) | |
tree | de6ade85f9d825f1300f06eda4a7e1627c676d64 /refs.c | |
parent | 6af926e8bcab15b42faeee9f2fe1c69b5fea1b15 (diff) |
lockfile.c: make lock_file return a meaningful errno on failurei
Making errno when returning from lock_file() meaningful, which should
fix
* an existing almost-bug in lock_ref_sha1_basic where it assumes
errno==ENOENT is meaningful and could waste some work on retries
* an existing bug in repack_without_refs where it prints
strerror(errno) and picks advice based on errno, despite errno
potentially being zero and potentially having been clobbered by
that point
Signed-off-by: Ronnie Sahlberg <sahlberg@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Acked-by: Michael Haggerty <mhagger@alum.mit.edu>
Diffstat (limited to 'refs.c')
-rw-r--r-- | refs.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -2212,6 +2212,7 @@ static int write_packed_entry_fn(struct ref_entry *entry, void *cb_data) return 0; } +/* This should return a meaningful errno on failure */ int lock_packed_refs(int flags) { struct packed_ref_cache *packed_ref_cache; |