[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn2rss.py patch

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: 2006-06-09 06:21:50 CEST

Bhuvaneswaran wrote:
>> Thanks Bhuvan! I've committed a modified version of your changes in
>> r20004. Please take a look at let me know what you think.
>
>
> The changes you did make this script look better. Thank you!
>
>> Support for ATOM seems like a laudable goal to me (though doesn't
>> necessarily match the script's name). What's the reasoning for
>> wanting to add support for older versions of RSS (e.g. do many clients
>> not yet offer support for RSS 2.0)?
>
>
> I wanted to support all type of RSS feeds. As you say, i understand it
> doesn't make more sense and we can add support for ATOM 1.0 alone. The
> current script name doesn't match it, so, as Malcolm suggested, i'll
> add a new script "svn2atom.py".

Hrm. Seems like all the feed types would have the same basic input needs.
Why not just rename the script to svnsyndicate.py or somesuch?

-- 
C. Michael Pilato <cmpilato@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on Fri Jun 9 06:22:20 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.