[m-rev.] for review: add RSS feed to Mercury site

Ian MacLarty maclarty at csse.unimelb.edu.au
Tue Feb 10 15:46:15 AEDT 2009


On Tue, Feb 10, 2009 at 11:09 AM, Paul Bone <pbone at csse.unimelb.edu.au> wrote:
> On Mon, Feb 09, 2009 at 05:07:56PM +1100, Ian MacLarty wrote:
>> For review by anyone.
>>
>> Estimated hours taken: 1
>>
>> Add RSS feed to Mercury site.
>>
>> Makefile:
>> Makefile.common:
>>       Add rules to build and install the rss.xml file.
>>
>> rss.php3:
>>       Build the RSS feed from the news database.
>>
>> include/functions.inc:
>>       Add a function to format a news item as an RSS
>>       item.
>>
>> include/index.inc:
>>       Add a link to the RSS feed from the home page.
>>
>> news/newsdb.inc:
>>       Mention the new feature in the news.
>>
>> Index: include/index.inc
>> ===================================================================
>> RCS file: /home/mercury1/repository/w3/include/index.inc,v
>> retrieving revision 1.5
>> diff -u -r1.5 index.inc
>> --- include/index.inc 13 Nov 1998 04:21:47 -0000      1.5
>> +++ include/index.inc 9 Feb 2009 05:58:50 -0000
>> @@ -15,3 +15,7 @@
>>  For older news items, see the
>>  (<a href="<? echo $root ?>/news.html">complete news archive</a>).
>>
>> +<p>
>> +
>> +News is also available as an RSS feed at
>> +<a href=\"http://www.mercury.csse.unimelb.edu.au/rss.xml\">http://www.mercury.csse.unimelb.edu.au/rss.xml</a>.
>
> This should be a relative link rather than an absolute link.  (this is
> not always easy to do in an included file).
>

No, I don't think it should be, because people will likely want to
copy and past this url into their feed aggregators.

Ian.
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list