> Two problems here:
>
> 1) It would be a lot safer to base the insertion point on the first
> <entry>, not the position of <author>. <author> doesn't carry any useful
> information in the feeds we generate, and in fact I'm considering
> removing it.
>
> 2) Do not use parentheses around an if condition. It's very abnormal
> Python style.
Please find attached the patch for these two problems. IMO, even though
<author> doesn't carry any useful information in the feeds, the feed
readers *may* expect this tag to validate the feed file. So, i prefer to
retain it.
[[
Patch by: Bhuvaneswaran Arumugam <bhuvan@collab.net>
* contrib/hook-scripts/svn2feed.py
(Svn2Atom.add_revision_item): Locate the first "entry" element and
insert the new element before that element. If the "entry" element
does not exist, then append the new element to the end of feed file.
Do not use parantheses around the "if" condition.
]]
--
Regards,
Bhuvaneswaran
Received on Sat Jul 22 05:11:41 2006