Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the SVN-Notify CPAN distribution.

Report information
The Basics
Id: 39193
Status: stalled
Priority: 0/
Queue: SVN-Notify

People
Owner: Nobody in particular
Requestors: th [...] outnow.ch
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: html mail output scrambled when using against subversion 1.5.x
Date: Fri, 12 Sep 2008 00:32:47 +0200
To: bug-SVN-Notify [...] rt.cpan.org
From: "Thomas Hägi [outnow.ch]" <th [...] outnow.ch>
SVN::Notify v2.77 Perl v5.8.8 (ActivePerl win32 build) Windows 2003 Server Subversion v1.5.2 hi there since we upgraded our svn installation from 1.4.x to the latest 1.5 version, svnnotify's html-output is scrambled - the diff always appear above (i.e. outside) the modified file box: --8<-------------------------- <div id="patch"> <h3>Diff</h3> <del>--- main/www/_scripts/code.js 2008-09-11 18:19:26 UTC (rev 670) </del><ins>+++ main/www/_scripts/code.js 2008-09-11 22:08:39 UTC (rev 671) </ins><span class="lines">@@ -205,10 +205,8 @@ </span><span class="cx"> type = 'Ajax'; </span><span class="cx"> } </span><span class="cx"> </span><ins>+ pageTracker._trackEvent(type, event, path); </ins><span class="cx"> </span><del>- var eventTracker = getTracker()._createEventTracker(type); - eventTracker._trackEvent(event, path); - </del><span class="cx"> if (path.substr(path.length - 1, 1) != '/') </span><span class="cx"> path += '/'; </span><span class="cx"> </span><a id="mainwww_scriptscodejs"></a> <div class="modfile"><h4>Modified: main/www/_scripts/code.js ( => )</h4> <pre class="diff"><span> <span class="info"> </span></span></pre> </div> --------------------------->8-- please notice how the <div id="patch"> is outside <pre class="diff">, where it really ought to be... please let me know if more information about the setup is needed. cheers, thomas
I do not use Subversion anymore myself, so am not in a position to test and fix this. If anyone has the tuits, please feel free to fork, fix, and submit a pull request here: https://github.com/theory/svn-notify/ --David