unofficial mirror of meta@public-inbox.org
 help / color / mirror / Atom feed
* [PATCH] git: local_nick: handle trailing or redundant '/' in git_dir
@ 2021-03-29 23:58 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2021-03-29 23:58 UTC (permalink / raw)
  To: meta

Some cgit configs use trailing slashes in pathnames
which we preserve internally.

Before this change, trailing slashes in cgit config files
was causing ViewVCS (SolverGit) output to show up as "???"
for coderepos without cgitUrl configured.
---
 lib/PublicInbox/Git.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/PublicInbox/Git.pm b/lib/PublicInbox/Git.pm
index 2ae5eff9..96ac17a3 100644
--- a/lib/PublicInbox/Git.pm
+++ b/lib/PublicInbox/Git.pm
@@ -426,7 +426,7 @@ sub local_nick ($) {
 	my ($self) = @_;
 	my $ret = '???';
 	# don't show full FS path, basename should be OK:
-	if ($self->{git_dir} =~ m!/([^/]+)(?:/\.git)?\z!) {
+	if ($self->{git_dir} =~ m!/([^/]+)(?:/*\.git/*)?\z!) {
 		$ret = "$1.git";
 	}
 	wantarray ? ($ret) : $ret;

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2021-03-29 23:58 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-29 23:58 [PATCH] git: local_nick: handle trailing or redundant '/' in git_dir Eric Wong

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).