all repos — cgit @ 6857bec50a52340fa6b85d626f49d45dd331ed0e

a hyperfast web frontend for git written in c

ui-tree.c: add support for path-selected submodule links

The current 'repo.module-link' option is sufficient when all gitlinks
in a repository can be converted to commit links in a uniform way, but
not when different submodules/paths needs different settings.

This patch adds support for 'repo.module-link.<path>', which will be
used for linking to submodules at paths matching one such entry.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Lars Hjemli hjemli@gmail.com
Wed, 15 Jun 2011 10:04:13 +0200
commit

6857bec50a52340fa6b85d626f49d45dd331ed0e

parent

8729d251a900b2e6e22cc4c93a2193fd8a2b9acf

7 files changed, 78 insertions(+), 10 deletions(-)

jump to
M cgit.ccgit.c

@@ -60,6 +60,8 @@ static void process_cached_repolist(const char *path);

void repo_config(struct cgit_repo *repo, const char *name, const char *value) { + struct string_list_item *item; + if (!strcmp(name, "name")) repo->name = xstrdup(value); else if (!strcmp(name, "clone-url"))

@@ -86,7 +88,10 @@ else if (!strcmp(name, "max-stats"))

repo->max_stats = cgit_find_stats_period(value, NULL); else if (!strcmp(name, "module-link")) repo->module_link= xstrdup(value); - else if (!strcmp(name, "section")) + else if (!prefixcmp(name, "module-link.")) { + item = string_list_append(&repo->submodules, name + 12); + item->util = xstrdup(value); + } else if (!strcmp(name, "section")) repo->section = xstrdup(value); else if (!strcmp(name, "readme") && value != NULL) repo->readme = xstrdup(value);

@@ -465,6 +470,7 @@ cgit_print_error(fmt("Invalid branch: %s", tmp));

cgit_print_docend(); return 1; } + sort_string_list(&ctx->repo->submodules); cgit_prepare_repo_env(ctx->repo); return 0; }
M cgit.hcgit.h

@@ -88,6 +88,7 @@ time_t mtime;

struct cgit_filter *about_filter; struct cgit_filter *commit_filter; struct cgit_filter *source_filter; + struct string_list submodules; }; typedef void (*repo_config_fn)(struct cgit_repo *repo, const char *name,
M cgitrc.5.txtcgitrc.5.txt

@@ -417,6 +417,12 @@ submodule is printed in a directory listing. The arguments for the

formatstring are the path and SHA1 of the submodule commit. Default value: <module-link> +repo.module-link.<path>:: + Text which will be used as the formatstring for a hyperlink when a + submodule with the specified subdirectory path is printed in a + directory listing. The only argument for the formatstring is the SHA1 + of the submodule commit. Default value: none. + repo.max-stats:: Override the default maximum statistics period. Valid values are equal to the values specified for the global "max-stats" setting. Default
M shared.cshared.c

@@ -71,6 +71,7 @@ ret->about_filter = ctx.cfg.about_filter;

ret->commit_filter = ctx.cfg.commit_filter; ret->source_filter = ctx.cfg.source_filter; ret->clone_url = ctx.cfg.clone_url; + ret->submodules.strdup_strings = 1; return ret; }
M ui-shared.cui-shared.c

@@ -503,6 +503,62 @@ name = fmt("%s %s...", typename(obj->type), shortrev);

reporevlink(page, name, NULL, NULL, ctx.qry.head, fullrev, NULL); } +struct string_list_item *lookup_path(struct string_list *list, + const char *path) +{ + struct string_list_item *item; + + while (path && path[0]) { + if ((item = string_list_lookup(list, path))) + return item; + if (!(path = strchr(path, '/'))) + break; + path++; + } + return NULL; +} + +void cgit_submodule_link(const char *class, char *path, const char *rev) +{ + struct string_list *list; + struct string_list_item *item; + char tail, *dir; + size_t len; + + tail = 0; + list = &ctx.repo->submodules; + item = lookup_path(list, path); + if (!item) { + len = strlen(path); + tail = path[len - 1]; + if (tail == '/') { + path[len - 1] = 0; + item = lookup_path(list, path); + } + } + html("<a "); + if (class) + htmlf("class='%s' ", class); + html("href='"); + if (item) { + html_attr(fmt(item->util, rev)); + } else if (ctx.repo->module_link) { + dir = strrchr(path, '/'); + if (dir) + dir++; + else + dir = path; + html_attr(fmt(ctx.repo->module_link, dir, rev)); + } else { + html("#"); + } + html("'>"); + html_txt(path); + html("</a>"); + if (item && tail) + path[len - 1] = tail; +} + void cgit_print_date(time_t secs, const char *format, int local_time) { char buf[64];
M ui-shared.hui-shared.h

@@ -51,6 +51,9 @@ extern void cgit_self_link(char *name, const char *title,

const char *class, struct cgit_context *ctx); extern void cgit_object_link(struct object *obj); +extern void cgit_submodule_link(const char *class, char *path, + const char *rev); + extern void cgit_print_error(const char *msg); extern void cgit_print_date(time_t secs, const char *format, int local_time); extern void cgit_print_age(time_t t, time_t max_relative, const char *format);
M ui-tree.cui-tree.c

@@ -150,13 +150,7 @@ html("<tr><td class='ls-mode'>");

cgit_print_filemode(mode); html("</td><td>"); if (S_ISGITLINK(mode)) { - htmlf("<a class='ls-mod' href='"); - html_attr(fmt(ctx.repo->module_link, - name, - sha1_to_hex(sha1))); - html("'>"); - html_txt(name); - html("</a>"); + cgit_submodule_link("ls-mod", fullpath, sha1_to_hex(sha1)); } else if (S_ISDIR(mode)) { cgit_tree_link(name, NULL, "ls-dir", ctx.qry.head, curr_rev, fullpath);

@@ -177,8 +171,9 @@ fullpath, 0, NULL, NULL, ctx.qry.showmsg);

if (ctx.repo->max_stats) cgit_stats_link("stats", NULL, "button", ctx.qry.head, fullpath); - cgit_plain_link("plain", NULL, "button", ctx.qry.head, curr_rev, - fullpath); + if (!S_ISGITLINK(mode)) + cgit_plain_link("plain", NULL, "button", ctx.qry.head, curr_rev, + fullpath); html("</td></tr>\n"); free(name); return 0;