<p dir="ltr">Herbert now seems to have applied the patch. Hopefully this will improve the email situation for everyone. </p>
<p dir="ltr">Matt</p>
<div class="gmail_quote">On 20 Jan 2016 1:15 pm, "Matthew Pickering" <<a href="mailto:matthewtpickering@gmail.com">matthewtpickering@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I attach a patch which should improve the notifications.<br>
<br>
Here is an example of the new output.<br>
<br>
<a href="https://www.irccloud.com/pastebin/HdlVbQt4/" rel="noreferrer" target="_blank">https://www.irccloud.com/pastebin/HdlVbQt4/</a><br>
<br>
Matt<br>
<br>
On Tue, Jan 19, 2016 at 11:29 AM, Matthew Pickering<br>
<<a href="mailto:matthewtpickering@gmail.com">matthewtpickering@gmail.com</a>> wrote:<br>
> Have you had any chance to think about this yet Herbert?<br>
><br>
> Matt<br>
><br>
> On Wed, Jan 6, 2016 at 6:44 PM, Ben Gamari <<a href="mailto:ben@smart-cactus.org">ben@smart-cactus.org</a>> wrote:<br>
>> Ben Gamari <<a href="mailto:ben@smart-cactus.org">ben@smart-cactus.org</a>> writes:<br>
>><br>
>>> Matthew Pickering <<a href="mailto:matthewtpickering@gmail.com">matthewtpickering@gmail.com</a>> writes:<br>
>>><br>
>>>> I subscribe to the ghc-tickets[1] mailing list which provides updates<br>
>>>> to all trac tickets. This is very useful, however when a ticket<br>
>>>> description is updated the whole description is included in the email<br>
>>>> which makes it hard to see what actually changed. The web interface<br>
>>>> shows a nice diff[2] of the changes, I think it would be good if<br>
>>>> emails could also include a diff rather than the current quite useless<br>
>>>> output.<br>
>>>><br>
>>> I think this would be a great improvement. I, for one, am quite guilty<br>
>>> of incrementally editing ticket descriptions and the current email<br>
>>> notifications are nearly useless in this case.<br>
>>><br>
>>>> After a bit of investigation, it appears that the easiest way to<br>
>>>> achieve this is apply a simple patch to our copy of trac. The current<br>
>>>> format is hard coded on line 558 in this module, it seems easy to<br>
>>>> modify this section to instead provide a diff.<br>
>>>><br>
>>> Herbert, perhaps we could do something along these lines?<br>
>>><br>
>> Ping.<br>
>><br>
>> Cheers,<br>
>><br>
>> - Ben<br>
</blockquote></div>