Closed Bug 1022563 Opened 7 years ago Closed 6 years ago

Don't use --enable-trace-malloc on SeaMonkey debug desktop tinderbox builds

Categories

(SeaMonkey :: Build Config, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philip.chee, Assigned: ewong)

References

Details

Attachments

(1 file)

+++ 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
Attachment #8436825 - Flags: review?(bugspam.Callek) → review+
(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)
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?
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?
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?
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)
All three trees have this pushed.  c-c (and c-a from uplift), c-b and recently
c-r.

Closing
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.