[PATCH] link in gitweb rss feed

[PATCH] link in gitweb rss feed

[Erik van Konijnenburg]

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
@@ -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".

Re: [PATCH] link in gitweb rss feed

[Kay Sievers]

On Wed, Jun 15, 2005 at 05:41:48PM +0200, Erik van Konijnenburg wrote:
> 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