git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] gitweb: Remove extra "/" in path names for git_get_project_list

From
Aneesh Kumar K.V <aneesh.kumar@gmail.com>
Date
Nov 3, 2006, 05:11 UTC
Message-ID
<454ACF91.50601@gmail.com>

Without this change we get a wrong $pfxlen value and the check_export_ok() checks with with a wrong directory name. Without this patch the below $projects_list fails with gitweb

$projects_list = "/tmp/a/b/";
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
---
 gitweb/gitweb.perl |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 7710cc2..a0a9aaf 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -897,6 +897,8 @@ sub git_get_projects_list {
 	if (-d $projects_list) {
 		# search in directory
 		my $dir = $projects_list;
+		# remove the trailing "/"
+		$dir =~ s!/+$!!;
 		my $pfxlen = length("$dir");
 
 		File::Find::find({
-- 
1.4.3.3.gc954-dirty
Next: Junio C Hamano
Message 1 of 3 in “gitweb: Remove extra "/" in path names for git_get_project_list”
  1. gitweb: Remove extra "/" in path names for git_get_project_listAneesh Kumar K.V, Nov 3, 2006
  2. Junio C HamanoNov 3, 2006
  3. Jakub NarebskiNov 3, 2006

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.