diff options
| author | Johan Herland <johan@herland.net> | 2010-11-15 20:41:00 +0100 | 
|---|---|---|
| committer | Lars Hjemli <hjemli@gmail.com> | 2011-02-19 14:25:14 +0100 | 
| commit | df522794c38934be3229a11e0e2432a1f2a3bc8d (patch) | |
| tree | f11aef6d303a5327303a4471d47444764bea53d8 | |
| parent | scan_path(): Improve handling of inaccessible directories (diff) | |
| download | cgit-df522794c38934be3229a11e0e2432a1f2a3bc8d.tar.gz cgit-df522794c38934be3229a11e0e2432a1f2a3bc8d.tar.bz2 cgit-df522794c38934be3229a11e0e2432a1f2a3bc8d.zip | |
scan_path(): Do not recurse into hidden directories by default
Paths that start with a period ('.') are considered hidden in the Unix world.
scan_path() should arguably not recurse into these directories by default.
This patch makes it so, and introduces the "scan-hidden-path" config variable
for overriding the new default and revert to the old behaviour (scanning _all_
directories, including hidden .directories).
Signed-off-by: Johan Herland <johan@herland.net>
Signed-off-by: Lars Hjemli <larsh@prediktor.no>
Diffstat (limited to '')
| -rw-r--r-- | cgit.c | 3 | ||||
| -rw-r--r-- | cgit.h | 1 | ||||
| -rw-r--r-- | cgitrc.5.txt | 8 | ||||
| -rw-r--r-- | scan-tree.c | 2 | 
4 files changed, 14 insertions, 0 deletions
| @@ -195,6 +195,8 @@ void config_cb(const char *name, const char *value)  				      ctx.cfg.project_list, repo_config);  		else  			scan_tree(expand_macros(value), repo_config); +	else if (!strcmp(name, "scan-hidden-path")) +		ctx.cfg.scan_hidden_path = atoi(value);  	else if (!strcmp(name, "section-from-path"))  		ctx.cfg.section_from_path = atoi(value);  	else if (!strcmp(name, "source-filter")) @@ -315,6 +317,7 @@ static void prepare_context(struct cgit_context *ctx)  	ctx->cfg.robots = "index, nofollow";  	ctx->cfg.root_title = "Git repository browser";  	ctx->cfg.root_desc = "a fast webinterface for the git dscm"; +	ctx->cfg.scan_hidden_path = 0;  	ctx->cfg.script_name = CGIT_SCRIPT_NAME;  	ctx->cfg.section = "";  	ctx->cfg.summary_branches = 10; @@ -207,6 +207,7 @@ struct cgit_config {  	int noheader;  	int renamelimit;  	int remove_suffix; +	int scan_hidden_path;  	int section_from_path;  	int snapshots;  	int summary_branches; diff --git a/cgitrc.5.txt b/cgitrc.5.txt index 8e51ca5..1dc3cce 100644 --- a/cgitrc.5.txt +++ b/cgitrc.5.txt @@ -269,6 +269,14 @@ root-title::  	Text printed as heading on the repository index page. Default value:  	"Git Repository Browser". +scan-hidden-path:: +	If set to "1" and scan-path is enabled, scan-path will recurse into +	directories whose name starts with a period ('.'). Otherwise, +	scan-path will stay away from such directories (considered as +	"hidden"). Note that this does not apply to the ".git" directory in +	non-bare repos. This must be defined prior to scan-path. +	Default value: 0. See also: scan-path. +  scan-path::  	A path which will be scanned for repositories. If caching is enabled,  	the result will be cached as a cgitrc include-file in the cache diff --git a/scan-tree.c b/scan-tree.c index eda8c67..627af1b 100644 --- a/scan-tree.c +++ b/scan-tree.c @@ -183,6 +183,8 @@ static void scan_path(const char *base, const char *path, repo_config_fn fn)  				continue;  			if (ent->d_name[1] == '.' && ent->d_name[2] == '\0')  				continue; +			if (!ctx.cfg.scan_hidden_path) +				continue;  		}  		buf = malloc(strlen(path) + strlen(ent->d_name) + 2);  		if (!buf) { | 
