Closed
Bug 696788
Opened 13 years ago
Closed 13 years ago
Build using jemalloc and SDK compiler crashes shortly after startup
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: neil, Assigned: neil)
Details
Attachments
(1 file)
1015 bytes,
patch
|
khuey
:
review+
Callek
:
feedback+
|
Details | Diff | Splinter Review |
The CRT provided with the Windows SDK compiler doesn't import free from crtdll.obj, so we don't try to build our own private CRT import library. This means that when we enable jemalloc, we pass MOZ_UTILS_LDFLAGS directly to NSPR/NSS via DLLFLAGS. However there are two problems with this: In NSPR this doesn't work at all, since NSPR doesn't know what EXPAND_LIBNAME_PATH or LIBXUL_DIST are, and so the $(call EXPAND_LIBNAME_PATH,mozutils,$(LIBXUL_DIST)/lib) simply gets ignored. For NSS the mozilla/security/manager/Makefile.in exports DLLFLAGS into the environment. This does seem to correctly expand it. However for some reason some (or all?) of the DLLs still end up linking to the CRT free instead.
Assignee | ||
Comment 1•13 years ago
|
||
I found that inserting the lib using -DEFAULTLIB (which resembles the way we do it for broken crtdll.obj builds) does ensure that NSS links correctly, although we then have to port the fix to comm-central because for some reason its copy of the broken DLLFLAGS confuses NSS again.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #569095 -
Flags: review?(khuey)
Attachment #569095 -
Flags: feedback?(bugspam.Callek)
Attachment #569095 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 2•13 years ago
|
||
Pushed changeset 204bd66034e2 to mozilla-central.
Updated•13 years ago
|
Attachment #569095 -
Flags: feedback?(bugspam.Callek) → feedback+
Assignee | ||
Comment 3•13 years ago
|
||
Pushed changeset 16efbe29a2a2 to comm-central.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•