[darcs-users] darcsweb - A gitweb-alike web interface for darcs

VMiklos vmiklos at frugalware.org
Fri Sep 9 15:38:19 UTC 2005


Alberto Bertogli wrote:
>>hmm, one bug, but i'm not sure if it is related to the recent changes:
>>
>>if you view the 'commit' of a patch that contains (only?) replaces, then 
>>it will be shown as binary. that may be ok. but then if you go to the 
>>filediff on that page, an error occures as the 'f' parameter left blank
>>
>>for example: 
>>http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=setup;a=filediff;h=20050827205444-e2957-58eddfb46ddfbd5fe20b5fc5c3292941f99476b2.gz;f=
> 
> Yes, that's a nasty bug regarding replaces (that weren't handled at all).
> I've fixed it too and now replaces should be shown properly (I've only
> tested with your repository so far, but I think it works =).
> 
> What I have noticed is that the darcs_filediff doesn't filter files out,
> and that's because darcs doesnt.
> 
> For instance, if I do the following on your repo:
> darcs chan -s -p 'update my email address' 'src/setup.de'
> 
> Instead of showing only the replaces affecting src/setup.de it shows the
> whole patch; the same goes for the xmlized output which I use.
> 
> However, the "darcs diff" output works just fine. Do you have any idea if
> this is expected behaviour or a bug in darcs?

i would suspect the later, but i'm not sure. i've never used -p and a 
filename togother

>>>PS: before somebody asks, I know some people would like the darcs format
>>>to be the default (I certanly don't). I will see in the future if I can
>>>make it configurable, but the code as-is makes this a bit tricky and
>>>implementing it will require some cleanup and reorganization.
>>
>>and what about a 'darcs_commitdiff' link after the commit and commitdiff 
>>ones?
> 
> I don't want to clutter the interface too much, there are already too many
> links in the navigation bar to add another one.
> 
> What I've done is add the small patch formats to the "commit" page, so you
> can jump to the darcs diff right from there if you want to.

ok, that's enough :)

udv / greetings,
VMiklos

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




More information about the darcs-users mailing list