* [PATCH] cgit.c: use resolve_ref() to guess_defbranch()
@ 2011-06-20 23:37 larsh
2011-06-20 23:39 ` Fwd: " hjemli
0 siblings, 1 reply; 2+ messages in thread
From: larsh @ 2011-06-20 23:37 UTC (permalink / raw)
The resolve_ref() function handles reading of git- and filesystem
symbolic links (including proper whitespace trimming) and packed refs.
There's no point in reimplementing this function in cgit.
Signed-off-by: Lars Hjemli <hjemli at gmail.com>
--
After deploying the wip-branch on hjemli.net, I noticed that the
default branch in the git.git repository wasn't correctly set. This was
due the HEAD symbolic ref having two trailing newlines, so I started
looking into how the git code handled this case and ended up with this
patch.
diff --git a/cgit.c b/cgit.c
index e3fbbf4..624cb2c 100644
--- a/cgit.c
+++ b/cgit.c
@@ -418,33 +418,13 @@ char *find_default_branch(struct cgit_repo *repo)
static char *guess_defbranch(const char *repo_path)
{
- int fd, len;
- char buffer[256];
- char *ref_start;
- char *head;
-
- head = fmt("%s/HEAD", repo_path);
- fd = open(head, O_RDONLY);
- if (fd == -1)
- return xstrdup("master");
-
- memset(buffer, 0, sizeof(buffer));
- len = read_in_full(fd, buffer, sizeof(buffer) - 1);
- close(fd);
-
- if(!memcmp(buffer, "ref: refs/heads/", 16))
- return xstrndup(buffer + 16, len - 17);
-
- if(strlen(buffer) == 41) {
- /* probably contains a SHA1 sum */
- memset(buffer, 0, sizeof(buffer));
- if(readlink(head, buffer, sizeof(buffer)-1)) {
- ref_start = memmem(buffer, sizeof(buffer)-1, "refs/heads/", 11);
- if(ref_start)
- return xstrdup(ref_start+11);
- }
- }
- return xstrdup("master");
+ const char *ref;
+ unsigned char sha1[20];
+
+ ref = resolve_ref("HEAD", sha1, 0, NULL);
+ if (!ref || prefixcmp(ref, "refs/heads/"))
+ return "master";
+ return xstrdup(ref + 11);
}
static int prepare_repo_cmd(struct cgit_context *ctx)
^ permalink raw reply [flat|nested] 2+ messages in thread
* Fwd: [PATCH] cgit.c: use resolve_ref() to guess_defbranch()
2011-06-20 23:37 [PATCH] cgit.c: use resolve_ref() to guess_defbranch() larsh
@ 2011-06-20 23:39 ` hjemli
0 siblings, 0 replies; 2+ messages in thread
From: hjemli @ 2011-06-20 23:39 UTC (permalink / raw)
[Resend since I failed to CC Julius, sorry for the noise]
---------- Forwarded message ----------
From: <larsh at hjemli.net>
Date: Tue, Jun 21, 2011 at 01:37
Subject: [PATCH] cgit.c: use resolve_ref() to guess_defbranch()
To: cgit at hjemli.net
The resolve_ref() function handles reading of git- and filesystem
symbolic links (including proper whitespace trimming) and packed refs.
There's no point in reimplementing this function in cgit.
Signed-off-by: Lars Hjemli <hjemli at gmail.com>
--
After deploying the wip-branch on hjemli.net, I noticed that the
default branch in the git.git repository wasn't correctly set. This was
due the HEAD symbolic ref having two trailing newlines, so I started
looking into how the git code handled this case and ended up with this
patch.
diff --git a/cgit.c b/cgit.c
index e3fbbf4..624cb2c 100644
--- a/cgit.c
+++ b/cgit.c
@@ -418,33 +418,13 @@ char *find_default_branch(struct cgit_repo *repo)
?static char *guess_defbranch(const char *repo_path)
?{
- ? ? ? int fd, len;
- ? ? ? char buffer[256];
- ? ? ? char *ref_start;
- ? ? ? char *head;
-
- ? ? ? head = fmt("%s/HEAD", repo_path);
- ? ? ? fd = open(head, O_RDONLY);
- ? ? ? if (fd == -1)
- ? ? ? ? ? ? ? return xstrdup("master");
-
- ? ? ? memset(buffer, 0, sizeof(buffer));
- ? ? ? len = read_in_full(fd, buffer, sizeof(buffer) - 1);
- ? ? ? close(fd);
-
- ? ? ? if(!memcmp(buffer, "ref: refs/heads/", 16))
- ? ? ? ? ? ? ? return xstrndup(buffer + 16, len - 17);
-
- ? ? ? if(strlen(buffer) == 41) {
- ? ? ? ? ? ? ? /* probably contains a SHA1 sum */
- ? ? ? ? ? ? ? memset(buffer, 0, sizeof(buffer));
- ? ? ? ? ? ? ? if(readlink(head, buffer, sizeof(buffer)-1)) {
- ? ? ? ? ? ? ? ? ? ? ? ref_start = memmem(buffer, sizeof(buffer)-1,
"refs/heads/", 11);
- ? ? ? ? ? ? ? ? ? ? ? if(ref_start)
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? return xstrdup(ref_start+11);
- ? ? ? ? ? ? ? }
- ? ? ? }
- ? ? ? return xstrdup("master");
+ ? ? ? const char *ref;
+ ? ? ? unsigned char sha1[20];
+
+ ? ? ? ref = resolve_ref("HEAD", sha1, 0, NULL);
+ ? ? ? if (!ref || prefixcmp(ref, "refs/heads/"))
+ ? ? ? ? ? ? ? return "master";
+ ? ? ? return xstrdup(ref + 11);
?}
?static int prepare_repo_cmd(struct cgit_context *ctx)
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-06-20 23:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-06-20 23:37 [PATCH] cgit.c: use resolve_ref() to guess_defbranch() larsh
2011-06-20 23:39 ` Fwd: " hjemli
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).