[PATCH] link in gitweb rss feed

Erik van Konijnenburg· Jun 15, 2005, 15:41 UTC · lore

The following patch makes the site name in an RSS feedreader for a gitweb project refer to the summary page for the the project, the same place where you picked up the feed in the first place. This seems more consistent than linking to the overview of all projects where the link used to up. Changed the link in OPML feed accordingly; this used to end up in the full log rather than the summary.

Patch was made against version 220 as shipped in Debian, and applies (with offset) to your version 221.

Regards, Erik

--- gitorg.cgi 2005-06-15 17:11:51.000000000 +0200 +++ gitweb.cgi 2005-06-15 17:16:42.000000000 +0200

Show changes to diff +2 −2

@@ -1227,7 +1227,7 @@
 	      "<rss version=\"2.0\" xmlns:content=\"http://purl.org/rss/1.0/modules/content/\">\n";
 	print "<channel>\n";
 	print "<title>$project</title>\n".
-	      "<link>" . escapeHTML("$my_url/$project/log") . "</link>\n".
+	      "<link>" . escapeHTML("$my_url?p=$project;a=summary") . "</link>\n".
 	      "<description>$project log</description>\n".
 	      "<language>en</language>\n";
 
@@ -1280,7 +1280,7 @@
 
 		my $path = escapeHTML(chop_str($proj{'path'}, 25, 5));
 		my $rss =  "$my_url?p=$proj{'path'};a=rss";
-		my $html =  "$my_url?p=$proj{'path'};a=log";
+		my $html =  "$my_url?p=$proj{'path'};a=summary";
 		print "<outline type=\"rss\" text=\"$path\" title=\"$path\" xmlUrl=\"$rss\" htmlUrl=\"$html\"/>\n";
 	}
 	print "</outline>\n".

Kay Sievers· Jun 22, 2005, 09:25 UTC · re: Erik van Konijnenburg · lore

Re: [PATCH] link in gitweb rss feed

On Wed, Jun 15, 2005 at 05:41:48PM +0200, Erik van Konijnenburg wrote:

Show 10 quoted lines

> The following patch makes the site name in an RSS
> feedreader for a gitweb project refer to the summary
> page for the the project, the same place where you picked
> up the feed in the first place.  This seems more consistent
> than linking to the overview of all projects where the
> link used to up.  Changed the link in OPML feed accordingly;
> this used to end up in the full log rather than the summary.
> 
> Patch was made against version 220 as shipped in Debian,
> and applies (with offset) to your version 221.

Should be installed with the last update on kernel.org.

Thanks, Kay

patchlink in gitweb rss feed