about summary refs log tree commit diff
diff options
context:
space:
mode:
authorJason A. Donenfeld <Jason@zx2c4.com>2012-07-11 04:24:01 +0200
committerJason A. Donenfeld <Jason@zx2c4.com>2012-07-12 20:01:46 +0200
commitb56be4ba3a942dd1978fe4bfecd9afc35aab0027 (patch)
tree1f193f16b3b54e92d1a23a42ed3a9eced13acc0f
parentcss: only use div#cgit (diff)
downloadcgit-pink-b56be4ba3a942dd1978fe4bfecd9afc35aab0027.tar.gz
cgit-pink-b56be4ba3a942dd1978fe4bfecd9afc35aab0027.zip
scan-tree: Support gitweb.description.
Use gitweb.description instead of description file to determine
description, if option is enabled.
-rw-r--r--cgit.c3
-rw-r--r--cgit.h1
-rw-r--r--cgitrc.5.txt7
-rw-r--r--scan-tree.c24
4 files changed, 28 insertions, 7 deletions
diff --git a/cgit.c b/cgit.c
index b9b3a66..ec5bbce 100644
--- a/cgit.c
+++ b/cgit.c
@@ -163,6 +163,8 @@ void config_cb(const char *name, const char *value)
 		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 @@ static void prepare_context(struct cgit_context *ctx)
 	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;
diff --git a/cgit.h b/cgit.h
index 6ee6769..f4d0e52 100644
--- a/cgit.h
+++ b/cgit.h
@@ -199,6 +199,7 @@ struct cgit_config {
 	int embedded;
 	int enable_filter_overrides;
 	int enable_gitweb_owner;
+	int enable_gitweb_desc;
 	int enable_http_clone;
 	int enable_index_links;
 	int enable_commit_graph;
diff --git a/cgitrc.5.txt b/cgitrc.5.txt
index a72241f..86a19a9 100644
--- a/cgitrc.5.txt
+++ b/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.
diff --git a/scan-tree.c b/scan-tree.c
index 378d795..3d4e417 100644
--- a/scan-tree.c
+++ b/scan-tree.c
@@ -48,19 +48,24 @@ static int is_git_dir(const char *path)
 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)
 {
 	while (from >= s && *from != c)
@@ -89,8 +94,9 @@ static void add_repo(const char *base, const char *path, repo_config_fn fn)
 		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 @@ static void add_repo(const char *base, const char *path, repo_config_fn fn)
 	}
 	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);