Closed
Bug 462740
Opened 16 years ago
Closed 16 years ago
Building Spidermonkey (js/src) with -j3 fails
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: Biesinger, Assigned: benjamin)
References
Details
Attachments
(1 file)
(deleted),
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
$ make -C js/src clean
$ make -C js/src -j3
...
link -NOLOGO -DLL -OUT:js3250.dll -PDB:js3250.pdb -SUBSYSTEM:WINDOWS jsapi.obj
jsarena.obj jsarray.obj jsatom.obj jsbool.obj jscntxt.obj jsdate.obj jsdbgapi.ob
j jsdhash.obj jsdtoa.obj jsemit.obj jsexn.obj jsfun.obj jsgc.obj jshash.obj jsin
terp.obj jsinvoke.obj jsiter.obj jslock.obj jslog2.obj jslong.obj jsmath.obj jsn
um.obj jsobj.obj json.obj jsopcode.obj jsparse.obj jsprf.obj jsregexp.obj jsscan
.obj jsscope.obj jsscript.obj jsstr.obj jsutil.obj jsxdrapi.obj jsxml.obj prmjti
me.obj jstracer.obj Assembler.obj Fragmento.obj LIR.obj RegAlloc.obj avmplus.obj
Nativei386.obj jsbuiltins.obj js3240.res -MANIFESTUAC:NO -NXCOMPAT -SAFESEH
-DYNAMICBASE -DEBUG -DEBUGTYPE:CV c:/moz/builds/hg-dbg/dist/lib/nspr4.li
b c:/moz/builds/hg-dbg/dist/lib/plc4.lib c:/moz/builds/hg-dbg/dist/lib/plds4.lib
kernel32.lib user32.lib gdi32.lib winmm.lib wsock32.lib advapi32.lib
link -NOLOGO -OUT:js.exe -PDB:js3250.pdb -MANIFESTUAC:NO -NXCOMPAT -SAFESEH -DY
NAMICBASE -DEBUG -DEBUGTYPE:CV js.obj jsapi.obj jsarena.obj jsarray.obj
jsatom.obj jsbool.obj jscntxt.obj jsdate.obj jsdbgapi.obj jsdhash.obj jsdtoa.obj
jsemit.obj jsexn.obj jsfun.obj jsgc.obj jshash.obj jsinterp.obj jsinvoke.obj js
iter.obj jslock.obj jslog2.obj jslong.obj jsmath.obj jsnum.obj jsobj.obj json.ob
j jsopcode.obj jsparse.obj jsprf.obj jsregexp.obj jsscan.obj jsscope.obj jsscrip
t.obj jsstr.obj jsutil.obj jsxdrapi.obj jsxml.obj prmjtime.obj jstracer.obj Asse
mbler.obj Fragmento.obj LIR.obj RegAlloc.obj avmplus.obj Nativei386.obj jsbuilti
ns.obj js3240.res c:/moz/builds/hg-dbg/dist/lib/nspr4.lib c:/moz/builds/hg-dbg/d
ist/lib/plc4.lib c:/moz/builds/hg-dbg/dist/lib/plds4.lib kernel32.lib user32.l
ib gdi32.lib winmm.lib wsock32.lib advapi32.lib
Bibliothek "js3250.lib" und Objekt Bibliothek "js.lib" und Objekt "j"js32s
.exp"50 werden erstellt.
.exp" werden erstellt.
LINK : fatal error LNK1318: Unexpected PDB error: ACCESS_DENIED (15) "c:\moz\builds\hg-dbg\js\src\js3250.pdb".
Reporter | ||
Comment 1•16 years ago
|
||
(this is with VC9 express on windows vista)
Comment 2•16 years ago
|
||
Yeah, I had the same thing (VC9 on Vista). Happened the first time (clean build with -j3) but went fine the next, still with -j3.
Reporter | ||
Updated•16 years ago
|
Summary: Building spidermonkey with -j3 fails → Building Spidermonkey (js/src) with -j3 fails
Assignee | ||
Updated•16 years ago
|
Assignee: general → benjamin
Comment 3•16 years ago
|
||
Same here with the identical configuration. It worked fine on a second try yesterday. But now it stopped again after a hg pull and hg update.
Assignee | ||
Comment 4•16 years ago
|
||
This patch uses a single (constant-named, even) PDB file for all compiled files, and a separate PDB file is generated by the linker for the final DLL or program.
Attachment #346326 -
Flags: review?(ted.mielczarek)
Comment 5•16 years ago
|
||
Comment on attachment 346326 [details] [diff] [review]
Use a separate PDB for compiled files and for the final library/program, rev. 1
# make builds on Windows. Now we just write to a pdb file per compiled file.
# See bug 286179 <https://bugzilla.mozilla.org/show_bug.cgi?id=286179> for
# details. -- chase@mozilla.org
Can you kill this outdated and no-longer-correct comment?
Otherwise looks fine. Do you want to make the same COMPILE/LINK distinction for HOST_PDBFILE?
Attachment #346326 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 6•16 years ago
|
||
Pushed http://hg.mozilla.org/mozilla-central/rev/5d832205b7fb
The distinction mainly exists for performance reasons, and since host-compiles are uncommon we can just leave them in the slow path (separate PDB for each object file).
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Flags: in-testsuite-
Flags: in-litmus-
You need to log in
before you can comment on or make changes to this bug.
Description
•