diff options
| author | Sergey Vlasov <vsu@altlinux.ru> | 2005-09-23 16:28:33 +0400 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2005-09-23 14:30:45 -0700 |
| commit | e2b77f026a251a47ebdf634107e76f7b457087af (patch) | |
| tree | 087a6bf8af105edb27a222540fd0ce8bd1398008 /local-fetch.c | |
| parent | 1a951815ddaa4e4b570cc67e204f45e9a12841e0 (diff) | |
| download | git-e2b77f026a251a47ebdf634107e76f7b457087af.tar.gz | |
[PATCH] Fix "git-local-fetch -s" with packed source repository
"git-local-fetch -s" did not work with a packed repository, because
symlink() happily created a link to a non-existing object file,
therefore fetch_file() always returned success, and fetch_pack() was
not called. Fixed by calling stat() before symlink() to ensure the
file really exists.
Signed-off-by: Sergey Vlasov <vsu@altlinux.ru>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'local-fetch.c')
| -rw-r--r-- | local-fetch.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/local-fetch.c b/local-fetch.c index a3e35f9c81..6216c68a4c 100644 --- a/local-fetch.c +++ b/local-fetch.c @@ -65,9 +65,17 @@ static int copy_file(const char *source, const char *dest, const char *hex) return -1; } } - if (use_symlink && !symlink(source, dest)) { - pull_say("symlink %s\n", hex); - return 0; + if (use_symlink) { + struct stat st; + if (stat(source, &st)) { + fprintf(stderr, "cannot stat %s: %s\n", source, + strerror(errno)); + return -1; + } + if (!symlink(source, dest)) { + pull_say("symlink %s\n", hex); + return 0; + } } if (use_filecopy) { int ifd, ofd, status; |