Closed
Bug 1022563
Opened 9 years ago
Closed 9 years ago
Don't use --enable-trace-malloc on SeaMonkey debug desktop tinderbox builds
Categories
(SeaMonkey :: Build Config, defect)
SeaMonkey
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philip.chee, Assigned: ewong)
References
Details
Attachments
(1 file)
1.81 KB,
patch
|
Callek
:
review+
Callek
:
approval-comm-beta+
Callek
:
approval-comm-release+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1013014 +++ > On TBPL, debug desktop builds are built with --enable-trace-malloc. This means > they don't use jemalloc, because TraceMalloc is incompatible with jemalloc. But > as far as I know, the TraceMalloc enabling serves no purpose, other than perhaps > to provide compile coverage. See also Thunderbird Bug 1022449
![]() |
Assignee | |
Comment 1•9 years ago
|
||
Attachment #8436825 -
Flags: review?(bugspam.Callek)
Updated•9 years ago
|
Attachment #8436825 -
Flags: review?(bugspam.Callek) → review+
![]() |
Assignee | |
Comment 2•9 years ago
|
||
Pushed to comm-central: https://hg.mozilla.org/comm-central/rev/6a8f85bc2880
![]() |
||
Comment 3•9 years ago
|
||
Bug 1018039 might also be relevant.
![]() |
Assignee | |
Comment 4•9 years ago
|
||
(In reply to Nicholas Nethercote [:njn] from comment #3) > Bug 1018039 might also be relevant. Do we need to add --enable-dmd on our builders?
Flags: needinfo?(bugspam.Callek)
![]() |
Assignee | |
Comment 5•9 years ago
|
||
Comment on attachment 8436825 [details] [diff] [review] removed --enable-trace-malloc option from debug mozconfigs.(v1) [Approval Request Comment] Regression caused by (bug #): User impact if declined: no windows build. Testing completed (on m-c, etc.): Risk to taking this patch (and alternatives if risky): String changes made by this patch:
Attachment #8436825 -
Flags: approval-comm-beta?
Attachment #8436825 -
Flags: approval-comm-aurora?
![]() |
Assignee | |
Comment 6•9 years ago
|
||
Pushed to comm-beta: https://hg.mozilla.org/releases/comm-beta/rev/9e0bc6702db2
![]() |
Assignee | |
Comment 7•9 years ago
|
||
Comment on attachment 8436825 [details] [diff] [review] removed --enable-trace-malloc option from debug mozconfigs.(v1) It's already in comm-aurora(due to uplift) and comm-central.
Attachment #8436825 -
Flags: approval-comm-aurora?
![]() |
Assignee | |
Comment 8•9 years ago
|
||
Comment on attachment 8436825 [details] [diff] [review] removed --enable-trace-malloc option from debug mozconfigs.(v1) [Approval Request Comment] Regression caused by (bug #): User impact if declined: debug builds fail. Testing completed (on m-c, etc.): Risk to taking this patch (and alternatives if risky): String changes made by this patch:
Attachment #8436825 -
Flags: approval-comm-release?
Updated•9 years ago
|
Attachment #8436825 -
Flags: approval-comm-release?
Attachment #8436825 -
Flags: approval-comm-release+
Attachment #8436825 -
Flags: approval-comm-beta?
Attachment #8436825 -
Flags: approval-comm-beta+
Flags: needinfo?(bugspam.Callek)
![]() |
Assignee | |
Comment 9•9 years ago
|
||
Pushed to comm-release: https://hg.mozilla.org/releases/comm-release/rev/f29497549d0e
![]() |
Assignee | |
Comment 10•9 years ago
|
||
All three trees have this pushed. c-c (and c-a from uplift), c-b and recently c-r. Closing
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•