cgit

commit fb9e6d1594a24fe4e551fd57a9c91fd18b14806e

Author: Lars Hjemli <hjemli@gmail.com>

Merge branch 'jh/scan-path'

* jh/scan-path:
  scan_path(): Do not recurse into hidden directories by default
  scan_path(): Improve handling of inaccessible directories

 cgit.c | 3 +++
 cgit.h | 1 +
 cgitrc.5.txt | 8 ++++++++
 scan-tree.c | 20 +++++++++++---------


diff --git a/cgit.c b/cgit.c
index 53ab68da2d7ff8f8b54deafcfdb3de47cf0c89b0..71f3fc8b36fbc56d403d188dcce02172f502e780 100644
--- a/cgit.c
+++ b/cgit.c
@@ -199,6 +199,8 @@ 			scan_projects(expand_macros(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"))
@@ -319,6 +321,7 @@ 	ctx->cfg.remove_suffix = 0;
 	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;




diff --git a/cgit.h b/cgit.h
index bed770b11baffb6f5927ec3f3629fa7bf960088d..74aa340b1014d1cebb965202d29638d0236356ac 100644
--- a/cgit.h
+++ b/cgit.h
@@ -210,6 +210,7 @@ 	int noplainemail;
 	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 3c20fe1a075f88da8c51a5ab790d2ce20c43d02f..a83283073c04cd167e03d7deb9b5eeca22f2f4f3 100644
--- a/cgitrc.5.txt
+++ b/cgitrc.5.txt
@@ -274,6 +274,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 a0e09ceac4212c1f8a07ca391d6bc67d6b0cb438..627af1b23240dbb8401d61d8e897afaf0eb05ca4 100644
--- a/scan-tree.c
+++ b/scan-tree.c
@@ -159,30 +159,31 @@ }
 
 static void scan_path(const char *base, const char *path, repo_config_fn fn)
 {
-	DIR *dir;
+	DIR *dir = opendir(path);
 	struct dirent *ent;
 	char *buf;
 	struct stat st;
 
+	if (!dir) {
+		fprintf(stderr, "Error opening directory %s: %s (%d)\n",
+			path, strerror(errno), errno);
+		return;
+	}
 	if (is_git_dir(path)) {
 		add_repo(base, path, fn);
-		return;
+		goto end;
 	}
 	if (is_git_dir(fmt("%s/.git", path))) {
 		add_repo(base, fmt("%s/.git", path), fn);
-		return;
-	}
-	dir = opendir(path);
-	if (!dir) {
-		fprintf(stderr, "Error opening directory %s: %s (%d)\n",
-			path, strerror(errno), errno);
-		return;
+		goto end;
 	}
 	while((ent = readdir(dir)) != NULL) {
 		if (ent->d_name[0] == '.') {
 			if (ent->d_name[1] == '\0')
 				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);
@@ -202,6 +203,7 @@ 		if (S_ISDIR(st.st_mode))
 			scan_path(base, buf, fn);
 		free(buf);
 	}
+end:
 	closedir(dir);
 }