Closed Bug 392882 Opened 17 years ago Closed 17 years ago

ActionMonkey: Firefox build is broken

Categories

(Firefox Build System :: General, defect)

Other Branch
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jorendorff, Assigned: jorendorff)

References

Details

Attachments

(1 file)

MMgc isn't in the include path for all the modules that include "jscntxt.h". It's not as simple as copying all the MMgc headers into ${OBJDIR}/dist/include/js. Those headers must always be included with the same set of MMgc-related defines, especially in debug mode; otherwise you can have linker problems or just crashes.
Blocks: 393023
bsmedberg suggests that these flags should go wherever JS_THREADSAFE goes, which makes sense.
Attached patch v1 (deleted) — Splinter Review
Turned out to be slightly more work than that. (I ran across bug 394447 while doing this.)
Assignee: nobody → jorendorff
Status: NEW → ASSIGNED
Attachment #279103 - Flags: review?(benjamin)
Attachment #279103 - Flags: review?(benjamin) → review+
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: