{"thread":{"id":"954","subject":"[PATCH] link in gitweb rss feed","startedAt":"2005-06-15T15:41:48Z","lastAt":"2005-06-22T09:25:13Z","messageCount":2,"participants":["Erik van Konijnenburg","Kay Sievers"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"4978","messageId":"20050615174148.C3099@banaan.localdomain","threadId":"954","inReplyTo":null,"subject":"[PATCH] link in gitweb rss feed","fromName":"Erik van Konijnenburg","fromEmail":"ekonijn@xs4all.nl","sentAt":"2005-06-15T15:41:48Z","receivedAt":"2005-06-15T15:41:48Z","isPatch":true,"sender":{"key":"ekonijn@xs4all.nl","avatar":null},"body":"The following patch makes the site name in an RSS\nfeedreader for a gitweb project refer to the summary\npage for the the project, the same place where you picked\nup the feed in the first place.  This seems more consistent\nthan linking to the overview of all projects where the\nlink used to up.  Changed the link in OPML feed accordingly;\nthis used to end up in the full log rather than the summary.\n\nPatch was made against version 220 as shipped in Debian,\nand applies (with offset) to your version 221.\n\nRegards,\nErik\n\n--- gitorg.cgi\t2005-06-15 17:11:51.000000000 +0200\n+++ gitweb.cgi\t2005-06-15 17:16:42.000000000 +0200\n@@ -1227,7 +1227,7 @@\n \t      \"<rss version=\\\"2.0\\\" xmlns:content=\\\"http://purl.org/rss/1.0/modules/content/\\\">\\n\";\n \tprint \"<channel>\\n\";\n \tprint \"<title>$project</title>\\n\".\n-\t      \"<link>\" . escapeHTML(\"$my_url/$project/log\") . \"</link>\\n\".\n+\t      \"<link>\" . escapeHTML(\"$my_url?p=$project;a=summary\") . \"</link>\\n\".\n \t      \"<description>$project log</description>\\n\".\n \t      \"<language>en</language>\\n\";\n \n@@ -1280,7 +1280,7 @@\n \n \t\tmy $path = escapeHTML(chop_str($proj{'path'}, 25, 5));\n \t\tmy $rss =  \"$my_url?p=$proj{'path'};a=rss\";\n-\t\tmy $html =  \"$my_url?p=$proj{'path'};a=log\";\n+\t\tmy $html =  \"$my_url?p=$proj{'path'};a=summary\";\n \t\tprint \"<outline type=\\\"rss\\\" text=\\\"$path\\\" title=\\\"$path\\\" xmlUrl=\\\"$rss\\\" htmlUrl=\\\"$html\\\"/>\\n\";\n \t}\n \tprint \"</outline>\\n\".\n"},{"id":"5115","messageId":"20050622092513.GA7629@vrfy.org","threadId":"954","inReplyTo":"20050615174148.C3099@banaan.localdomain","subject":"Re: [PATCH] link in gitweb rss feed","fromName":"Kay Sievers","fromEmail":"kay.sievers@vrfy.org","sentAt":"2005-06-22T09:25:13Z","receivedAt":"2005-06-22T09:25:13Z","isPatch":true,"sender":{"key":"kay.sievers@vrfy.org","avatar":null},"body":"On Wed, Jun 15, 2005 at 05:41:48PM +0200, Erik van Konijnenburg wrote:\n> The following patch makes the site name in an RSS\n> feedreader for a gitweb project refer to the summary\n> page for the the project, the same place where you picked\n> up the feed in the first place.  This seems more consistent\n> than linking to the overview of all projects where the\n> link used to up.  Changed the link in OPML feed accordingly;\n> this used to end up in the full log rather than the summary.\n> \n> Patch was made against version 220 as shipped in Debian,\n> and applies (with offset) to your version 221.\n\nShould be installed with the last update on kernel.org.\n\nThanks,\nKay\n"}]}