# [PATCH] gitweb: Do not automatically append " git" to custom site name

4 messages from 2006-10-11 to 2006-10-24. Participants: Petr Baudis, Jakub Narebski, Jeff King.
Thread: https://gitlist.dev/t/5890

## Petr Baudis, 2006-10-11 20:22

Subject: [PATCH] gitweb: Do not automatically append " git" to custom site name
Message-ID: <20061011202228.27279.97423.stgit@rover>
URL: https://gitlist.dev/e/20061011202228.27279.97423.stgit%40rover

```
If you customized the site name, you probably do not want the " git"
appended so that the page title is not bastardized; I want repo.or.cz pages
titled "Public Git Hosting", not "Public Git Hosting git" (what's hosting
what?).

This slightly changes the $site_name semantics but only very
insignificantly.

Signed-off-by: Petr Baudis <pasky@suse.cz>
---

 gitweb/gitweb.perl |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index c2dfdab..d165fdc 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -39,7 +39,7 @@ our $home_link_str = "++GITWEB_HOME_LINK
 
 # name of your site or organization to appear in page titles
 # replace this with something more descriptive for clearer bookmarks
-our $site_name = "++GITWEB_SITENAME++" || $ENV{'SERVER_NAME'} || "Untitled";
+our $site_name = "++GITWEB_SITENAME++" || "$ENV{'SERVER_NAME'} Git" || "Untitled Git";
 
 # filename of html text to include at top of each page
 our $site_header = "++GITWEB_SITE_HEADER++";
@@ -1455,7 +1455,7 @@ sub git_header_html {
 	my $status = shift || "200 OK";
 	my $expires = shift;
 
-	my $title = "$site_name git";
+	my $title = "$site_name";
 	if (defined $project) {
 		$title .= " - $project";
 		if (defined $action) {
@@ -3842,7 +3842,7 @@ sub git_opml {
 <?xml version="1.0" encoding="utf-8"?>
 <opml version="1.0">
 <head>
-  <title>$site_name Git OPML Export</title>
+  <title>$site_name OPML Export</title>
 </head>
 <body>
 <outline text="git RSS feeds">

```

## Jakub Narebski, 2006-10-11 20:31

Subject: Re: [PATCH] gitweb: Do not automatically append " git" to custom site name
Message-ID: <egjkbc$ht$1@sea.gmane.org>
URL: https://gitlist.dev/e/egjkbc%24ht%241%40sea.gmane.org
In-Reply-To: <20061011202228.27279.97423.stgit@rover>

```
Petr Baudis wrote:

> +our $site_name = "++GITWEB_SITENAME++" || "$ENV{'SERVER_NAME'} Git" || "Untitled Git";

I'd use perhaps "[$ENV{'SERVER_NAME'}] Git".
-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git

```

## Jeff King, 2006-10-11 22:06

Subject: Re: [PATCH] gitweb: Do not automatically append " git" to custom site name
Message-ID: <20061011220621.GA4041@coredump.intra.peff.net>
URL: https://gitlist.dev/e/20061011220621.GA4041%40coredump.intra.peff.net
In-Reply-To: <20061011202228.27279.97423.stgit@rover>

```
On Wed, Oct 11, 2006 at 10:22:28PM +0200, Petr Baudis wrote:

> +our $site_name = "++GITWEB_SITENAME++" || "$ENV{'SERVER_NAME'} Git" || "Untitled Git";

This will never show "Untitled" (instead it shows " Git") and will cause
a perl warning if $ENV{SERVER_NAME} is not defined and warnings are
enabled (which tye are in gitweb).

I think this is what you want:

our $site_name = "++GITWEB_SITENAME++" ||
                 ($ENV{SERVER_NAME} || "Untitled") . " Git";

-Peff

```

## Petr Baudis, 2006-10-24 03:18

Subject: [PATCH] gitweb: Do not automatically append " git" to custom site name
Message-ID: <20061024031838.3323.14282.stgit@rover>
URL: https://gitlist.dev/e/20061024031838.3323.14282.stgit%40rover
In-Reply-To: <20061011220621.GA4041@coredump.intra.peff.net>

```
If you customized the site name, you probably do not want the " git"
appended so that the page title is not bastardized; I want repo.or.cz pages
titled "Public Git Hosting", not "Public Git Hosting git" (what's hosting
what?).

This slightly changes the $site_name semantics but only very
insignificantly.

Signed-off-by: Petr Baudis <pasky@suse.cz>
---

 gitweb/gitweb.perl |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index e77fc11..1e80f43 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -39,7 +39,8 @@ our $home_link_str = "++GITWEB_HOME_LINK
 
 # name of your site or organization to appear in page titles
 # replace this with something more descriptive for clearer bookmarks
-our $site_name = "++GITWEB_SITENAME++" || $ENV{'SERVER_NAME'} || "Untitled";
+our $site_name = "++GITWEB_SITENAME++"
+                 || ($ENV{'SERVER_NAME'} || "Untitled") . " Git";
 
 # filename of html text to include at top of each page
 our $site_header = "++GITWEB_SITE_HEADER++";
@@ -1433,7 +1434,7 @@ sub git_header_html {
 	my $status = shift || "200 OK";
 	my $expires = shift;
 
-	my $title = "$site_name git";
+	my $title = "$site_name";
 	if (defined $project) {
 		$title .= " - $project";
 		if (defined $action) {
@@ -3847,7 +3848,7 @@ sub git_opml {
 <?xml version="1.0" encoding="utf-8"?>
 <opml version="1.0">
 <head>
-  <title>$site_name Git OPML Export</title>
+  <title>$site_name OPML Export</title>
 </head>
 <body>
 <outline text="git RSS feeds">

```
