aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJunio C Hamano <junkio@cox.net>2005-10-05 15:29:48 -0700
committerJunio C Hamano <junkio@cox.net>2005-10-05 15:29:48 -0700
commit4cfc63db1daa0ebdb284ae6972928e41a317a0c8 (patch)
treec9f67fa052f7946c0e21a1e0c15b6a6616891592
parentf73ae1fc5d5308fecf0a4bdbd7312f7f5b00df55 (diff)
downloadgit-4cfc63db1daa0ebdb284ae6972928e41a317a0c8.tar.gz
git-4cfc63db1daa0ebdb284ae6972928e41a317a0c8.tar.xz
clone-pack: use create_symref() instead of raw symlink.
This was the last instance of symlink() in coreish part. Signed-off-by: Junio C Hamano <junkio@cox.net>
-rw-r--r--clone-pack.c24
1 files changed, 16 insertions, 8 deletions
diff --git a/clone-pack.c b/clone-pack.c
index 49820c657..c102ca885 100644
--- a/clone-pack.c
+++ b/clone-pack.c
@@ -51,6 +51,7 @@ static void write_refs(struct ref *ref)
struct ref *head = NULL, *head_ptr, *master_ref;
char *head_path;
+ /* Upload-pack must report HEAD first */
if (!strcmp(ref->name, "HEAD")) {
head = ref;
ref = ref->next;
@@ -60,17 +61,21 @@ static void write_refs(struct ref *ref)
while (ref) {
if (is_master(ref))
master_ref = ref;
- if (head && !memcmp(ref->old_sha1, head->old_sha1, 20)) {
- if (!head_ptr || ref == master_ref)
- head_ptr = ref;
- }
+ if (head &&
+ !memcmp(ref->old_sha1, head->old_sha1, 20) &&
+ !strncmp(ref->name, "refs/heads/",11) &&
+ (!head_ptr || ref == master_ref))
+ head_ptr = ref;
+
write_one_ref(ref);
ref = ref->next;
}
- if (!head)
+ if (!head) {
+ fprintf(stderr, "No HEAD in remote.\n");
return;
+ }
- head_path = git_path("HEAD");
+ head_path = strdup(git_path("HEAD"));
if (!head_ptr) {
/*
* If we had a master ref, and it wasn't HEAD, we need to undo the
@@ -82,6 +87,7 @@ static void write_refs(struct ref *ref)
unlink(head_path);
}
write_one_ref(head);
+ free(head_path);
return;
}
@@ -89,13 +95,15 @@ static void write_refs(struct ref *ref)
if (master_ref)
return;
+ fprintf(stderr, "Setting HEAD to %s\n", head_ptr->name);
+
/*
* Uhhuh. Other end didn't have master. We start HEAD off with
* the first branch with the same value.
*/
- unlink(head_path);
- if (symlink(head_ptr->name, head_path) < 0)
+ if (create_symref(head_path, head_ptr->name) < 0)
die("unable to link HEAD to %s", head_ptr->name);
+ free(head_path);
}
static int clone_pack(int fd[2], int nr_match, char **match)