Closed
Bug 287064
Opened 20 years ago
Closed 20 years ago
Replace Mail::Mailer / Mail::Header with Mail::Box
Categories
(Bugzilla :: Email Notifications, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: glob, Assigned: glob)
References
()
Details
Attachments
(1 file, 2 obsolete files)
5.14 KB,
patch
|
bugreport
:
review-
|
Details | Diff | Splinter Review |
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•20 years ago
|
||
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
Comment 2•20 years ago
|
||
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]
Comment 3•20 years ago
|
||
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?
Comment 4•20 years ago
|
||
I hate to lose testfile.
Can we just put a log-to-file mechanism in right with this?
Comment 5•20 years ago
|
||
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?
Updated•20 years ago
|
Attachment #178637 -
Flags: review?
Comment 6•20 years ago
|
||
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-
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?
Comment 10•20 years ago
|
||
I think we have a new record for number of questions asked by the PAN Makefile.
Comment 11•20 years ago
|
||
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-
Comment 12•20 years ago
|
||
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•20 years ago
|
||
In my case...
1) I did let it install some of the optional components, and it failed.
2) This exposed bug 287926
Comment 14•20 years ago
|
||
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
Assignee | ||
Comment 15•20 years ago
|
||
--> 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
Updated•19 years ago
|
Target Milestone: Bugzilla 2.20 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•