Skip Menu |

This queue is for tickets about the activitymail CPAN distribution.

Report information
The Basics
Id: 6632
Status: resolved
Priority: 0/
Queue: activitymail
People
Owner: dwheeler [...] cpan.org
Requestors: jamesb-cpan [...] comcast.net
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 1.22

History Show all quoted text
  Tue Jun 15 13:46:16 2004 Guest - Ticket created
Subject: Diffs aren't formatted in a <pre> tag in HTML mode
Example: ********************************* ... [snip] ... </ul> </body> </html> Revision Data ------------- Index: XXX.pm =================================================================== RCS file: /cvsroot/XXX.pm,v retrieving revision 1.14 retrieving revision 1.15 ... [snip] ... ********************************************************** The diffs occur after the closing </html> tag, which probably isn't valid HTML. Moreover, for my mail reader (Outlook 2k with ie5 -- admittedly it's not great, but it's what we're stuck with) the diff is still treated as regular HTML, and it's mashed together into one paragraph. As a temporary fix I added an opening <pre> tag at the end of the footer, which is a hack but at least makes the diff readable.
  Thu Jun 17 21:05:08 2004 Guest - Correspondence added
From: jamesb-cpan [...] comcast.net
I'd also like to add that "<" and ">" aren't converted into "&lt;" and "&gt;" in the diff in HTML mode.
  Thu Jun 17 21:11:36 2004 Guest - Correspondence added
From: jamesb-cpan [...] comcast.net
[guest - Thu Jun 17 21:05:08 2004]: Show quoted text
> I'd also like to add that "<" and ">" aren't converted into "&lt;" and > "&gt;" in the diff in HTML mode.
Ugh. Apparently RT suffers from a similar bug, because it doesn't convert ampersand. So for clarification that's "[ampersand]lt;" and "[ampersand]gt;". Might want to make sure ampersand is quoted correctly in the diff output too.
  Thu Jun 17 21:19:42 2004 dwheeler [...] cpan.org - Correspondence added
Ack, damn HTML. As a workaround I suggest that you either use plain text or have the diff file attached to the message. I'm not sure when I'll have the tuits to work on activitymail again, given that I'm using Subversion more and more these days...
  Thu Jun 17 21:19:43 2004 dwheeler [...] cpan.org - Given to DWHEELER
  Thu Jun 17 21:19:43 2004 dwheeler [...] cpan.org - Status changed from 'new' to 'open'
  Mon Jun 20 12:35:17 2005 Guest - Correspondence added
From: moritz[AT]barsnick[DOT]net
I've just updated from 1.04 to 1.21 and like some of the new features. I had been patching 1.04 myself. ;-) [DWHEELER - Thu Jun 17 21:19:42 2004]: Show quoted text
> Ack, damn HTML. As a workaround I suggest that you either use plain > text or have the diff file attached to the message.
Oh, I was wondering. I noticed the documentation said to use either -c or -l (I used neither), and once I added -c, it was calling is_last_dir, () which only works if log_dir() was called previously, which is only the case _with_ -l. Hmm. Well, at least I'm glad I found this forum. I will disable -H for now. I'm still looking for a program with nicely formatted output, which doesn't attach the diffs. (cvsreport is very nice for daily summaries but doesn't include diffs in HTML mails.) Thanks for the tool, regards, Moritz
  Mon Jun 20 13:16:21 2005 david [...] kineticode.com - Correspondence added
From: David Wheeler <david [...] kineticode.com>
Subject: Re: [cpan #6632] Diffs aren't formatted in a <pre> tag in HTML mode
Date: Mon, 20 Jun 2005 10:16:08 -0700
To: bug-activitymail [...] rt.cpan.org
RT-Send-Cc:
On Jun 20, 2005, at 09:35 , Guest via RT wrote: Show quoted text
>> Ack, damn HTML. As a workaround I suggest that you either use plain >> text or have the diff file attached to the message.
Please try the attached patch. I haven't tested it, but it should be pretty close, if not dead-on. Be sure that you have HTML::Entities installed. Show quoted text
> Oh, I was wondering. I noticed the documentation said to use either -c > or -l (I used neither), and once I added -c, it was calling > is_last_dir, > () which only works if log_dir() was called previously, which is only > the case _with_ -l. Hmm.
Right. Use -l in the commitinfo file, and the use -c in the loginfo file. They depend on each other, but you can't use them in the same execution of the script. Show quoted text
> Well, at least I'm glad I found this forum. I will disable -H for now. > I'm still looking for a program with nicely formatted output, which > doesn't attach the diffs. (cvsreport is very nice for daily summaries > but doesn't include diffs in HTML mails.)
Let me know if the attached patch fixes the issue, and if so, I'll release 1.22. Best, David

Message body is not shown because sender requested not to inline it.

  Mon Jun 20 13:31:43 2005 david [...] kineticode.com - Correspondence added
From: David Wheeler <david [...] kineticode.com>
Subject: Re: [cpan #6632] Diffs aren't formatted in a <pre> tag in HTML mode
Date: Mon, 20 Jun 2005 10:31:31 -0700
To: bug-activitymail [...] rt.cpan.org
RT-Send-Cc:
On Jun 20, 2005, at 10:16 , David Wheeler via RT wrote: Show quoted text
> Let me know if the attached patch fixes the issue, and if so, I'll > release 1.22.
Ignore that last patch. I think that this one is closer to it. Best, David

Message body is not shown because sender requested not to inline it.

  Mon Nov 21 10:13:19 2005 dwheeler [...] cpan.org - Status changed from 'open' to 'resolved'
  Mon Nov 21 10:13:32 2005 dwheeler [...] cpan.org - Fixed in 1.22 added