[fossil-users] The fdiff page, 'patch' mode, has html

12 views
Skip to first unread message

Lluís Batlle i Rossell

unread,
Feb 22, 2012, 3:51:44 AM2/22/12
to Fossil ML
Hello,

the fdiff page, patch mode, has HTML in it:

One example:
http://fossil-scm.org/index.html/fdiff?v1=806340fbc989550a&v2=f1d6ecabf37517e4&patch

Regards,
Lluís
_______________________________________________
fossil-users mailing list
fossil...@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users

Lluís Batlle i Rossell

unread,
Feb 28, 2012, 11:11:10 AM2/28/12
to Fossil ML
On Wed, Feb 22, 2012 at 09:51:44AM +0100, Lluís Batlle i Rossell wrote:
> Hello,
>
> the fdiff page, patch mode, has HTML in it:
>
> One example:
> http://fossil-scm.org/index.html/fdiff?v1=806340fbc989550a&v2=f1d6ecabf37517e4&patch

Isn't this problem important for a fix? I just write as a reminder...

Regards,
Lluís.

Leo Razoumov

unread,
Mar 24, 2012, 4:39:52 PM3/24/12
to fossil...@lists.fossil-scm.org, fossi...@lists.fossil-scm.org, Dr. D. Richard Hipp, vi...@viric.name
2012/2/28 Lluís Batlle i Rossell <vi...@viric.name>:

> On Wed, Feb 22, 2012 at 09:51:44AM +0100, Lluís Batlle i Rossell wrote:
>> Hello,
>>
>> the fdiff page, patch mode, has HTML in it:
>>
>> One example:
>> http://fossil-scm.org/index.html/fdiff?v1=806340fbc989550a&v2=f1d6ecabf37517e4&patch
>
> Isn't this problem important for a fix? I just write as a reminder...
>

Yes, I do consider it an annoyance worth fixing. Please, find my patch
below. It prevents HTML tags from appearing in the plain text patch in
fdiff webpage. I cross-post this message to Richard and to the
fossil-dev list. Hopefully, the developers will find time to apply it.
The patch is against fossil trunk [8f496ed159].

--Leo--

fdiff.patch

Lluís Batlle i Rossell

unread,
Mar 24, 2012, 5:04:30 PM3/24/12
to Leo Razoumov, fossi...@lists.fossil-scm.org, fossil...@lists.fossil-scm.org

That patch is since some weeks in a branch I maintain with the annoyances I
consider worth fixing, and features worth having (as far as I can program).

http://fossil-scm.org/index.html/info/675b49efa3

I have not checked how similar is your patch to mine, but the effect should be
the same.

Reply all
Reply to author
Forward
0 new messages