Closed Bug 287064 Opened 20 years ago Closed 20 years ago

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

Categories

(Bugzilla :: Email Notifications, defect)

2.18
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: glob, Assigned: glob)

References

()

Details

Attachments

(1 file, 2 obsolete files)

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/
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.
Flags: blocking2.20+
Target Milestone: --- → Bugzilla 2.20
We'll want to relnote that we don't require any of the optional dependencies, if you install it from CPAN.
Keywords: relnote
Whiteboard: [relnote bug and comment 2]
Attached patch Patch v1 (obsolete) (deleted) — Splinter Review
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.
Attachment #178637 - Flags: review?
I hate to lose testfile. Can we just put a log-to-file mechanism in right with this?
Attached patch Patch v2 (obsolete) (deleted) — Splinter Review
As before, but with testfile support for Joel.
Assignee: bugzilla → colin.ogilvie
Attachment #178637 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #178641 - Flags: review?
Attachment #178637 - Flags: review?
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.
Assignee: colin.ogilvie → bugzilla
Status: ASSIGNED → NEW
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 on attachment 178641 [details] [diff] [review] Patch v2 this code doesn't catch any errors that may occur during message delivery (think smtp).
Attachment #178641 - Flags: review? → review-
Attached patch patch v3 (deleted) — Splinter Review
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
Attachment #178641 - Attachment is obsolete: true
Attachment #178720 - Flags: review?
I think we have a new record for number of questions asked by the PAN Makefile.
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??
Attachment #178720 - Flags: review? → review-
Depends on: 287926
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.
In my case... 1) I did let it install some of the optional components, and it failed. 2) This exposed bug 287926
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
--> 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.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → WONTFIX
Target Milestone: Bugzilla 2.20 → ---
Keywords: relnote
Whiteboard: [relnote bug and comment 2]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: