Bug #13836 for Test-Reporter: Forcing Net::SMTP in Test::Reporter and cpantest

Sat Jul 23 12:07:00 2005 wyant [...] cpan.org - Ticket created

Subject:

Forcing Net::SMTP in Test::Reporter and cpantest

Richard, I have found the attached mods to Test::Reporter and cpantest to be useful. The intent is to provide a way to tell Test::Reporter to use Net::SMTP even if Mail::Send is avaialable. The motivation for it, quite frankly, was laziness. I have a MacOS X system which is behind a NAT router. Surely it was I who installed Mail::Send (unless it's part of the core now, which seems unlikely), but I don't remember doing so. MacOS X has just enough of Postfix installed to try to send mail, but not enough to do anything useful, and I'm too lazy to try to sort out a working configuration for sending mail on a system I have no intent to use as a mail server. It was much easier to put the relevant two lines of code in Reporter.pm (not counting the POD), and three in cpantest to make use of them (not counting the "usage" lines). The attached tarball contains diff -u files for the relevant pieces/parts. I have taken the liberty to modify the version number, but that does NOT mean I assert the right to version your code - I was just trying to distinguish the modified code from the original from which I worked. What the diff files do is to give Test::Reporter an 'smtp' attribute. If this attribute is true, Net::SMTP is used to send mail, even if Mail::Send is installed. The cpantest script gets a corresponding -smtp option, with machinery to use it to set the smtp attribute of the Test::Reporter object. I dithered over the name of both attribute and method (the alternate being 'force_smtp'), but finally settled on the shorter one for my own use. I figured that if you accepted the change you would rename things if you thought it good, and in the meantime I would type less. Here's the canonical system information. Distribution: Test-Reporter-1.27 Perl version: v5.8.6 OS vendor/version: Darwin Wyants-White-iMac.local 7.9.0 Darwin Kernel Version 7.9.0: Wed Mar 30 20:11:17 PST 2005; root:xnu/xnu-517.12.7.obj~1/RELEASE_PPC Power Macintosh powerpc Thank you very much, Tom Wyant

Download Reporter-Mod.tar.gz
application/x-gzip 1.4k

Message body not shown because it is not plain text.

Tue May 23 23:45:47 2006 Guest - Correspondence added

From:

Rutger Vos

These patches fixed the issues (i.e. incomplete postfix configuration) on my OSX box. On Sat Jul 23 12:07:00 2005, WYANT wrote: Show quoted text

> Richard, > > I have found the attached mods to Test::Reporter and cpantest to be > useful. The intent is to provide a way to tell Test::Reporter to > use Net::SMTP even if Mail::Send is avaialable. The motivation for > it, quite frankly, was laziness. > > I have a MacOS X system which is behind a NAT router. Surely it was I > who installed Mail::Send (unless it's part of the core now, which > seems unlikely), but I don't remember doing so. MacOS X has just > enough of Postfix installed to try to send mail, but not enough to > do anything useful, and I'm too lazy to try to sort out a working > configuration for sending mail on a system I have no intent to use > as a mail server. It was much easier to put the relevant two lines > of code in Reporter.pm (not counting the POD), and three in > cpantest to make use of them (not counting the "usage" lines). > > The attached tarball contains diff -u files for the relevant > pieces/parts. I have taken the liberty to modify the version > number, but that does NOT mean I assert the right to version your > code - I was just trying to distinguish the modified code from the > original from which I worked. > > What the diff files do is to give Test::Reporter an 'smtp' attribute. > If this attribute is true, Net::SMTP is used to send mail, even if > Mail::Send is installed. The cpantest script gets a corresponding > -smtp option, with machinery to use it to set the smtp attribute of > the Test::Reporter object. I dithered over the name of both > attribute and method (the alternate being 'force_smtp'), but > finally settled on the shorter one for my own use. I figured that > if you accepted the change you would rename things if you thought > it good, and in the meantime I would type less. > > Here's the canonical system information. > > Distribution: Test-Reporter-1.27 > Perl version: v5.8.6 > OS vendor/version: Darwin Wyants-White-iMac.local 7.9.0 Darwin Kernel > Version 7.9.0: Wed Mar 30 20:11:17 PST 2005; root:xnu/xnu- > 517.12.7.obj~1/RELEASE_PPC Power Macintosh powerpc > > Thank you very much, > Tom Wyant

Tue May 23 23:45:48 2006 The RT System itself - Status changed from 'new' to 'open'

Wed Sep 20 00:37:25 2006 ANDK [...] cpan.org - Correspondence added

I'd like to endorse this feature request with a "me too" posting. I'd also need a way to disable Mail::Send in favor of Net::SMTP. I haven't yet figured out if it is a bug in Mail::Send or a misconfiguration of sendmail, but I know I can send mail via Net::SMTP but not via Mail::Send right now and a config variable to do so would make my life easier. Thanks!

Thu Sep 21 02:56:38 2006 FOX [...] cpan.org - Correspondence added

On Wed Sep 20 00:37:25 2006, ANDK wrote: Show quoted text

> I'd like to endorse this feature request with a "me too" posting. I'd > also need a way to disable Mail::Send in favor of Net::SMTP. I haven't > yet figured out if it is a bug in Mail::Send or a misconfiguration of > sendmail, but I know I can send mail via Net::SMTP but not via > Mail::Send right now and a config variable to do so would make my life > easier. Thanks!

Andreas, Hello. Thank you for your comment. I am working on this ticket right now and will have it completed on or before 01 October 2006.

Sat Apr 14 10:49:46 2007 FOX [...] cpan.org - Correspondence added

Migrated to: http://code.google.com/p/test-reporter/issues/detail?id=4

Sat Apr 14 10:49:53 2007 FOX [...] cpan.org - Status changed from 'open' to 'resolved'

Bug #13836 for Test-Reporter: Forcing Net::SMTP in Test::Reporter and cpantest

Preferred bug tracker