Don't define MOZ_JEMALLOC_IMPL when building webrtc-gtest

RESOLVED FIXED in Firefox 57

Status

RESOLVED FIXED
a year ago
8 months ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

unspecified
mozilla57

Firefox Tracking Flags

(firefox57 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
Comment hidden (mozreview-request)
Comment on attachment 8903382 [details]
Bug 1395769 - Don't define MOZ_JEMALLOC_IMPL when building webrtc-gtest.

https://reviewboard.mozilla.org/r/175218/#review180476

I could believe that this define needed to be set if we were linking in libxul code in and/or linking to mozglue (`MOZ_GLUE_IN_PROGRAM`?), but judging from this, it looks like we're not doing that?  Can't see why this would be needed in the current setup.
Attachment #8903382 - Flags: review?(nfroyd) → review+

Comment 3

a year ago
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/0e0bd516dd5f
Don't define MOZ_JEMALLOC_IMPL when building webrtc-gtest. r=froydnj
https://hg.mozilla.org/mozilla-central/rev/0e0bd516dd5f
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57

Updated

8 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.