Closed
Bug 436872
Opened 17 years ago
Closed 17 years ago
Build js shell on Windows in mozilla-central/tracemonkey
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 445243
People
(Reporter: bc, Unassigned)
References
Details
Attachments
(1 file)
1.09 KB,
patch
|
Details | Diff | Splinter Review |
The config/WINNT* files lack a definition for CXX which results in an attempt to compile cpp files using g++.
rules.mk has a typo where jsinterp.c is used to build jsinterp.obj.
With these errors fixed, the build still fails with:
js.cpp(2032) : error C2664: 'JS_ValueToInt32' : cannot convert parameter 3 from 'intN *' to 'int32 *'
Types pointed to are unrelated; conversion requires reinterpret_cast, C-style cast or function-style cast
Flags: in-testsuite-
Flags: in-litmus-
Comment 1•17 years ago
|
||
The patch looks good to me. I'll try it out on my Vista VM today.
That last error sounds like an Intel64 bug.
Reporter | ||
Comment 2•17 years ago
|
||
It was actually with a 32bit Windows XP VM although the host is 64bit.
Comment 3•17 years ago
|
||
Sorry, my Vista VM won't boot. I need to reinstall. If anyone else wants to take this review, please feel free.
Reporter | ||
Comment 4•17 years ago
|
||
from what i understand, actionmonkey is a dead end, but this is also applicable to tracemonkey which is not. Can I get an r+ or r- from someone?
Summary: Build js shell on Windows in mozilla-central/actionmonkey → Build js shell on Windows in mozilla-central/tracemonkey
Comment 5•17 years ago
|
||
"actionmonkey is a dead end" is not true, if we are doing anything like using an MMgc-style GC -- which I thought was the plan. If not, then XPCOMGC is a dead end and CC and ref-counting will live forever.
Please let's not throw away work we think we need to build on. Whether in the final analysis "actionmonkey" the repository dies, but patches from it live on in a more central repo, I do not care. The "actionmonkey is tamarin in SpiderMonkey clothing, via a big-bang integration after years of work" meme is dead, but that is not the same as "actionmonkey is a dead end". AM is needed for XPCOMGC, last I heard.
/be
Reporter | ||
Comment 6•17 years ago
|
||
Ok. Either I misremembered what I was told or someone misspoke. Which repositories should I care about? mozilla-central, tracemonkey, actionmonkey, what else? Do I need to file separate bugs on each repository? How do we track where the fixes have landed?
Comment 7•17 years ago
|
||
Jason: is actionmonkey on hiatus while you do DOM perf work for 3.1? Where is Jason doing jegc work?
/be
Updated•17 years ago
|
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → DUPLICATE
Comment 9•17 years ago
|
||
Yes, actionmonkey is on hiatus. I don't know where jegc is happening.
You need to log in
before you can comment on or make changes to this bug.
Description
•