Closed
Bug 146087
Opened 22 years ago
Closed 20 years ago
'sendmailnow' should be on by default.
Categories
(Bugzilla :: Email Notifications, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.16
People
(Reporter: CodeMachine, Assigned: goobix)
References
Details
(Whiteboard: [fixed in 2.16.x] [fixed in 2.18rc1])
Attachments
(2 files)
482 bytes,
patch
|
justdave
:
review+
|
Details | Diff | Splinter Review |
767 bytes,
patch
|
Details | Diff | Splinter Review |
I'm pretty sure we currently ship with the 'sendmailnow' parameter off. Given this is a performance enhancement most installations won't need, it can cause confusion (bug #141874), and problems with Sendmail clones, I don't see any reason we shouldn't ship with it being defaulted on.
Assignee | ||
Comment 1•21 years ago
|
||
Tested, applied againest the CVS tip, works as expected.
Assignee | ||
Updated•21 years ago
|
Attachment #128378 -
Flags: review?
Comment 2•21 years ago
|
||
Comment on attachment 128378 [details] [diff] [review] Sets the default of sendmailnow to 1. duh... ;)
Attachment #128378 -
Flags: review? → review+
Comment 3•21 years ago
|
||
-> patch author
Assignee: justdave → jocuri
Flags: approval+
Target Milestone: --- → Bugzilla 2.18
Comment 4•21 years ago
|
||
This should go in on the 2.16 branch ONLY. The patch for bug 84876 removes sendmailnow.
Assignee | ||
Updated•21 years ago
|
Status: NEW → ASSIGNED
Comment 5•21 years ago
|
||
Patch against the BUGZILLA-2_16-BRANCH.
Comment 6•21 years ago
|
||
/cvsroot/mozilla/webtools/bugzilla/defparams.pl,v <-- defparams.pl new revision: 1.73.2.6; previous revision: 1.73.2.5 done Also changed some bug information to be more accurate.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Component: Administration → Email Notifications
Resolution: --- → FIXED
Target Milestone: Bugzilla 2.18 → Bugzilla 2.16
Comment 7•20 years ago
|
||
*** Bug 239302 has been marked as a duplicate of this bug. ***
Comment 8•20 years ago
|
||
I want this on the trunk too. There's no reason this shouldn't have been checked in on the trunk. The Bug 84876 patch can still remove it when it's time, no matter what the setting.
Status: RESOLVED → REOPENED
Flags: blocking2.18+
Resolution: FIXED → ---
Updated•20 years ago
|
Whiteboard: [fixed in 2.16.x] [wanted for 2.18rc1]
Assignee | ||
Comment 9•20 years ago
|
||
Checking in defparams.pl; /cvsroot/mozilla/webtools/bugzilla/defparams.pl,v <-- defparams.pl new revision: 1.127; previous revision: 1.126 done
Status: REOPENED → RESOLVED
Closed: 21 years ago → 20 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•20 years ago
|
Whiteboard: [fixed in 2.16.x] [wanted for 2.18rc1] → [fixed in 2.16.x] [fixed in 2.18rc1]
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•