[darcs-users] darcsweb fails to generate rss feed for small repos

[VMiklos]

On Sun, Apr 23, 2006 at 05:52:18AM -0300, Alberto Bertogli <albertito at gmail.com> wrote:
> I can't reproduce this problem, I tried creating an empty repository and
> adding up to 5 patches, and it works well in every step (as it should,
> because the code just doesn't care how many patches are there).
> 
> What error do you get? Is the repo public, so I can download it and give
> it a try?

http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=frugalwareutils;a=rss

this fails

http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman-tools;a=rss

this works fine. the only difference i've found is the number of
patches. of course maybe i'm wrong :)

> And while we're at it: if anyone is aware of any regression/feature
> missing comparing to darcs.cgi, please let me know.

then here are two requests:

1) first search config in /etc/darcsweb/, this would make the packagers'
life a bit easier. something like:

sys.path.insert(0, '/etc/darcsweb/')

before the declaration of the empty configuration class

2) support for having cia links for the authors

$ diff -u darcsweb.cgi.orig darcsweb.cgi.cia
--- darcsweb.cgi.orig   2006-02-27 18:13:48.000000000 +0100
+++ darcsweb.cgi.cia    2006-03-08 10:14:39.000000000 +0100
@@ -1054,7 +1054,7 @@

                print """
   <td><i>%(age)s</i></td>
-  <td>%(author)s</td>
+  <td><a
href="http://cia.navi.cx/stats/author/%(href)s">%(author)s</a></td>
   <td>
     <a class="list" title="%(fullname)s"
href="%(myrname)s;a=commit;h=%(hash)s">
       <b>%(name)s</b>
@@ -1066,6 +1066,7 @@
   </td>
                """ % {
                        'age': how_old(p.local_date),
+                       'href': p.author,
                        'author': shorten_str(p.shortauthor, 26),
                        'myrname': config.myreponame,
                        'hash': p.hash,

of course it's just a quick&dirty hack, probably it could be disabled by
default and enabled with a config option (i thought i've already posted
it to here, but i can't find the patch in the archives)

udv / greetings,
VMiklos

-- 
Developer of Frugalware Linux, to make things frugal - http://frugalware.org