Closed Bug 826678 Opened 11 years ago Closed 11 years ago

Disable warnings about the deprecated Return::Value module when loading Email::Send

Categories

(Bugzilla :: Email Notifications, defect)

defect
Not set
major

Tracking

()

RESOLVED FIXED
Bugzilla 4.0

People

(Reporter: LpSolit, Assigned: LpSolit)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch patch, v1 (obsolete) — Splinter Review
As explained in bug 502625 comment 8, Return::Value 1.666002, which has been released last week, throws warnings every time this module is loaded about it being deprecated. All Bugzilla releases depend on Return::Value via Email::Send. Till bug 502625 is fixed (replace Email::Send by Email::Sender), we have to disable these warnings, else it's litterally polluting the web server error log (several tens of lines in the error log every time an email is sent!).

In the attached patch, I didn't patch Bugzilla::Send::Sendmail, despite it explicitly loads Return::Value, because this module is loaded by Bugzilla::Mailer which already contains $Return::Value::NO_CLUCK = 1.

I want to take this fix for 4.0.10 despite it's not a security bug, because the 4.0 branch will reach EOL at the end of 2013 at the earliest, and I suspect several admins will update their Perl modules meanwhile, including Return::Value. About the 3.6 branch, I doubt we will have another release on this branch as 4.4 final is almost there, and so this branch is very close to EOL.
Attachment #697896 - Flags: review?(glob)
Flags: blocking4.4+
Flags: blocking4.2.5+
Flags: blocking4.0.10+
Attached patch patch, v2Splinter Review
When checksetup.pl checks if Email::Send is installed, the same warning is displayed. So we have to disable the warning there too.
Attachment #697896 - Attachment is obsolete: true
Attachment #697896 - Flags: review?(glob)
Attachment #697925 - Flags: review?(glob)
Comment on attachment 697925 [details] [diff] [review]
patch, v2

Ewww, what nasty spewage. This patch does the trick for trunk but it'll need backports.
Attachment #697925 - Flags: review?(glob) → review+
Flags: approval?
(In reply to Teemu Mannermaa (:wicked) from comment #2)
> This patch does the trick for trunk but it'll need backports.

No backport needed. I just need to replace |use parent| by |use base| to fix the bitrot. I will do it on checkin.
Flags: approval?
Flags: approval4.4+
Flags: approval4.2+
Flags: approval4.0+
Flags: approval+
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/Mailer.pm
modified Bugzilla/Config/MTA.pm
modified Bugzilla/Install/Requirements.pm
Committed revision 8548.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.4/
modified Bugzilla/Mailer.pm
modified Bugzilla/Config/MTA.pm
modified Bugzilla/Install/Requirements.pm
Committed revision 8498.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.2/
modified Bugzilla/Mailer.pm
modified Bugzilla/Config/MTA.pm
modified Bugzilla/Install/Requirements.pm
Committed revision 8183.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.0/
modified Bugzilla/Mailer.pm
modified Bugzilla/Config/MTA.pm
modified Bugzilla/Install/Requirements.pm
Committed revision 7740.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.