Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Test-Reporter CPAN distribution.

Report information
The Basics
Id: 3264
Status: resolved
Priority: 0/
Queue: Test-Reporter

People
Owner: RSOD [...] cpan.org
Requestors: rurban [...] x-ray.at
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in:
  • 1.20
  • 1.21
  • 1.22
  • 1.23
  • 1.24
  • 1.25
Fixed in: (no value)



Subject: Test-Reporter-1.20 patch for cygwin
Since it's quite common that cygwin people don't have a mailer from Mail::Send installed, I recommend the following patch, so that the number of cygwin tests will increase. Me for example :) It will fail for cygwin anyway, since Mail::Send is not called the recommended way. About the mx stuff: I had to change my cpantest script to change the mx. I inserted: $Reporter->from('r.urban@inode.at (Reini Urban cpan:RURBAN)'); $Reporter->mx(['smtp.inode.at']);
--- Test-Reporter-1.20/lib/Test/Reporter.pm~ 2003-03-05 10:15:53.000000000 +0100 +++ Test-Reporter-1.20/lib/Test/Reporter.pm 2003-08-16 16:56:32.000000000 +0200 @@ -355,7 +355,7 @@ if ($recipient =~ /(?:perl|cpan)\.org$/) { push @tmprecipients, $recipient; } - else { + elsif ($self->mx->[0] =~ /perl.org/) { push @bad, $recipient; } } @@ -512,6 +512,7 @@ return $send if defined $send; return 0 if FAKE_NO_MAIL_SEND; + return 0 if $^O =~ /cygwin/; $send = eval {require Mail::Send}; }
[RURBAN - Sat Aug 16 11:18:26 2003]: Show quoted text
> Since it's quite common that cygwin people don't have a mailer from > Mail::Send installed, I recommend the following patch, so that the > number of cygwin tests will increase. Me for example :)
I see a variety of PASS results listed for MailTools under cygwin, including one for the latest version: http://www.nntp.perl.org/group/perl.cpan.testers/170831 I'll do some checking to see if these PASS results are false. Show quoted text
> It will fail for cygwin anyway, since Mail::Send is not called the > recommended way.
Thanks for the tip; I'll look into it. Show quoted text
> About the mx stuff: > I had to change my cpantest script to change the mx. > I inserted: > > $Reporter->from('r.urban@inode.at (Reini Urban cpan:RURBAN)'); > $Reporter->mx(['smtp.inode.at']);
I've updated the MX handling code inside of T:R 1.22 to support multiple MXs (instead of only using the first). I've also updated the address rejection code to only reject non-perl/cpan.org addresses when the SMTP connection is to a perl/cpan.org MX. Please let me know if 1.22 gives you any problems; it'll be on CPAN shortly. - R.
Test::Reporter 1.27 uses Net::SMTP on Windows now, instead of MailTools -- this should make a difference. Thanks for the report and patch. - R.