287064 - Replace Mail::Mailer / Mail::Header with Mail::Box

Status: RESOLVED WONTFIX

(Bugzilla :: Email Notifications, defect)

Description

[:glob ✱]

Mail::Mailer and Mail::Header are old modules that have issues with error
reporting and full compliance, and are not activly maintained anymore.

Mail::Box by far the best replacement.
See the docs at http://perl.overmeer.net/mailbox/html/

Comment 1

[Dave Miller [:justdave]]

We haven't yet had a release that contains this module as a prereq, and since
it's no longer maintained, it's in our best interest to replace it before we
release one that does.

Comment 2

[Max Kanat-Alexander]

We'll want to relnote that we don't require any of the optional dependencies, if
you install it from CPAN.

Comment 3

[Colin Ogilvie [:cso]]

Attachment: Patch v1

This adds Mail::Box support, instead of Mail::Mailer.

Note: This removes the "testfile" ability, unfortunately. I suspect we could
write our own module to do that, though.

Comment 4

[Joel Peshkin]

I hate to lose testfile.
Can we just put a log-to-file mechanism in right with this?

Comment 5

[Colin Ogilvie [:cso]]

Attachment: Patch v2

As before, but with testfile support for Joel.

Comment 6

[Colin Ogilvie [:cso]]

Glob,

Sorry - didn't mean to assign this to me - was a bit enthusiastic with the tabbing.

One thing that might need to be noted is that the "PPM" repository on landfill
will need the 2.060 module on it, or the requirement will need to be 2.059 for
the default ppm repository.

Comment 7

[:glob ✱]

please hold this bug -- i have already got an almost identical patch for this
issue, however i don't want to move to Mail::Box until i'm sure that it will
resolve the issues that i encountered in bug 126266 with Mail::Mailer.

Comment 8

[:glob ✱]

Comment on attachment 178641 [details] [diff] [review]
Patch v2

this code doesn't catch any errors that may occur during message delivery
(think smtp).

Comment 9

[:glob ✱]

Attachment: patch v3

i've finally got Mail::Box working with my utf8 bug, so here's an updated
patch, with the following changes:
  - min version dropped to 2.059 for windows (i haven't encountered any issues
    developing with this version)
  - exim added to list of supported mtas
  - multiple smtp server support
  - error reporting

Comment 10

[Joel Peshkin]

I think we have a new record for number of questions asked by the PAN Makefile.

Comment 11

[Joel Peshkin]

Comment on attachment 178720 [details] [diff] [review]
patch v3

I think this would be the only module we use that requires the user to use a
forced install.

We can't shift over to Mail::Box until the install is fixed.  The CPAN (2.060)
module, in addition to asking a zillion questions, hangs during its own tests
(Perl 5.8.0 from source).  This happened to both mkanat and me.

Let's find another option or wait until Mail::Box matures. (which, I really
hope it does -- it looks pretty neat)

Also...
} else {
+	     if ($method = 'sendmail' && !Param('sendmailnow')) {
+		 push @args, "-ODeliveryMode=deferred";

Should be ==, right??

Comment 12

[Colin Ogilvie [:cso]]

What problems did people have with the installation - it worked fine on my test
system - although I answered no to all the "optional" modules, and told it to
install dependancies.

Comment 13

[Joel Peshkin]

In my case...
1) I did let it install some of the optional components, and it failed.
2) This exposed bug 287926

Comment 14

[Joel Peshkin]

3) On a clean build of 5.8.1 from source with Bundle::CPAN installed, and
answering no to all of the optional components....

Perl itself hit a similar issue to bug 287926

43pop3/01basic.........NOK 1#     Failed test (43pop3/01basic.t at line 11)
#     Tried to use 'Mail::Transport::POP3'.
#     Error:  Insecure dependency in require while running with -T switch at
/home/newperl/perl/lib/5.8.1/i686-linux/IO/Socket.pm line 22.
# Compilation failed in require at ../lib/Mail/Transport/POP3.pm line 10.
# BEGIN failed--compilation aborted at ../lib/Mail/Transport/POP3.pm line 10.

and then hangs at...

Method socket() is not defined for a Mail::Transport::POP3.
43pop3/01basic.........ok 3/14  Mail::Reporter::AUTOLOAD('Mail::Transport::...')
called at 43pop3/01basic.t line 44

Comment 15

[:glob ✱]

--> wontfix

Mail::Header and Mail::Mailer have just been updated to resolve the issues i was
experiencing, so we no longer need to switch to Mail::Box.