scan-tree: Support gitweb.description. Use gitweb.description instead of description file to determine description, if option is enabled.
Jason A. Donenfeld Jason@zx2c4.com
Wed, 11 Jul 2012 04:24:01 +0200
4 files changed,
28 insertions(+),
7 deletions(-)
M
cgit.c
→
cgit.c
@@ -163,6 +163,8 @@ else if (!strcmp(name, "snapshots"))
ctx.cfg.snapshots = cgit_parse_snapshots_mask(value); else if (!strcmp(name, "enable-filter-overrides")) ctx.cfg.enable_filter_overrides = atoi(value); + else if (!strcmp(name, "enable-gitweb-desc")) + ctx.cfg.enable_gitweb_desc = atoi(value); else if (!strcmp(name, "enable-gitweb-owner")) ctx.cfg.enable_gitweb_owner = atoi(value); else if (!strcmp(name, "enable-http-clone"))@@ -336,6 +338,7 @@ ctx->cfg.cache_static_ttl = -1;
ctx->cfg.css = "/cgit.css"; ctx->cfg.logo = "/cgit.png"; ctx->cfg.local_time = 0; + ctx->cfg.enable_gitweb_desc = 1; ctx->cfg.enable_gitweb_owner = 1; ctx->cfg.enable_http_clone = 1; ctx->cfg.enable_tree_linenumbers = 1;
M
cgitrc.5.txt
→
cgitrc.5.txt
@@ -106,6 +106,13 @@ enable-filter-overrides::
Flag which, when set to "1", allows all filter settings to be overridden in repository-specific cgitrc files. Default value: none. +enable-gitweb-desc:: + If set to "1" and scan-path is enabled, we first check each repository + for the git config value "gitweb.description" to determine the owner. + Otherwise, the description is read from a file titled "description" + inside of the repository directory. + Default value: "1". See also: scan-path. + enable-gitweb-owner:: If set to "1" and scan-path is enabled, we first check each repository for the git config value "gitweb.owner" to determine the owner.
M
scan-tree.c
→
scan-tree.c
@@ -48,18 +48,23 @@
struct cgit_repo *repo; repo_config_fn config_fn; char *owner; +char *desc; static void repo_config(const char *name, const char *value) { config_fn(repo, name, value); } -static int git_owner_config(const char *key, const char *value, void *cb) +static int gitweb_config(const char *key, const char *value, void *cb) { - if (!strcmp(key, "gitweb.owner")) + if (ctx.cfg.enable_gitweb_owner && !strcmp(key, "gitweb.owner")) owner = xstrdup(value); + else if (ctx.cfg.enable_gitweb_desc && !strcmp(key, "gitweb.description")) + desc = xstrdup(value); return 0; } + + static char *xstrrchr(char *s, char *from, int c) {@@ -89,8 +94,9 @@ if (!stat(fmt("%s/noweb", path), &st))
return; owner = NULL; - if (ctx.cfg.enable_gitweb_owner) - git_config_from_file(git_owner_config, fmt("%s/config", path), NULL); + desc = NULL; + git_config_from_file(gitweb_config, fmt("%s/config", path), NULL); + if (base == path) rel = xstrdup(fmt("%s", path)); else@@ -118,9 +124,13 @@ owner = xstrdup(pwd->pw_gecos ? pwd->pw_gecos : pwd->pw_name);
} repo->owner = owner; - p = fmt("%s/description", path); - if (!stat(p, &st)) - readfile(p, &repo->desc, &size); + if (desc) + repo->desc = desc; + else { + p = fmt("%s/description", path); + if (!stat(p, &st)) + readfile(p, &repo->desc, &size); + } if (!repo->readme) { p = fmt("%s/README.html", path);