Closed
Bug 423692
Opened 17 years ago
Closed 17 years ago
Actionmonkey busted on Windows with NSPR flags and Tamarin defines
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: benjamin, Assigned: benjamin)
References
Details
Attachments
(2 files)
1.17 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
444 bytes,
patch
|
treilly
:
review+
|
Details | Diff | Splinter Review |
Actionmonkey is busted on Windows. There are three different bustages, two of which I can fix here and one of which is fixed upstream and just needs a merge.
I'm pretty sure the first one here works on the big-three platforms... it will need some more cleanup later for WINCE and perhaps a few others, although perhaps we should just fix nspr-config to do the right thing all the time.
Attachment #310293 -
Flags: review?(ted.mielczarek)
Assignee | ||
Comment 1•17 years ago
|
||
This suppresses a bogus compiler warning: class GC has members of type GCCondition, but since the constructors aren't inline you don't need GCCondition to be exported. I'm not sure why it's issuing the warning, but it should be safe to ignore.
It also export GCThread, which you do need to use MMgc.dll in threadsafe mode.
Attachment #310297 -
Flags: review?(treilly)
Updated•17 years ago
|
Attachment #310293 -
Flags: review?(ted.mielczarek) → review+
Comment 2•17 years ago
|
||
Please move the warning disable into winbuild.h with the other ones otherwise go ahead and push it.
Updated•17 years ago
|
Attachment #310297 -
Flags: review?(treilly) → review+
Assignee | ||
Comment 3•17 years ago
|
||
Patches pushed to actionmonkey and tamarin-central... trying to figure out whether to commit or merge to actionmonkey-tamarin.
With the latest m-c -> actionmonkey merge Jason is doing, this should be fixed.
Comment 4•17 years ago
|
||
Is this a dup of bug#428529?
Comment 5•17 years ago
|
||
In case it helps, today's builds are failing out with the following syntax error:
win32/build/dom/src/base/nsJSEnvironment.cpp
nsJSEnvironment.cpp
e:/builds/moz2-slave/actionmonkey-win32/build/dom/src/base/nsJSEnvironment.cpp(751) : error C2039: 'CompareString' : is not a member of 'nsICollation'
e:\builds\moz2-slave\actionmonkey-win32\build\obj-firefox\dist\include\locale\nsICollation.h(100) : see declaration of 'nsICollation'
e:/builds/moz2-slave/actionmonkey-win32/build/dom/src/base/nsJSEnvironment.cpp(3273) : warning C4242: '=' : conversion from 'PRBool' to 'PRPackedBool', possible loss of data
e:/builds/moz2-slave/actionmonkey-win32/build/dom/src/base/nsJSEnvironment.cpp(3292) : warning C4242: '=' : conversion from 'PRBool' to 'PRPackedBool', possible loss of data
e:/builds/moz2-slave/actionmonkey-win32/build/dom/src/base/nsJSEnvironment.cpp(3298) : warning C4242: '=' : conversion from 'PRBool' to 'PRPackedBool', possible loss of data
e:/builds/moz2-slave/actionmonkey-win32/build/dom/src/base/nsJSEnvironment.cpp(3876) : warning C4265: 'nsJSArgArray' : class has virtual functions, but destructor is not virtual
instances of this class may not be destructed correctly
make[6]: Leaving directory `/e/builds/moz2-slave/actionmonkey-win32/build/obj-firefox/dom/src/base'
make[5]: Leaving directory `/e/builds/moz2-slave/actionmonkey-win32/build/obj-firefox/dom/src'
make[6]: *** [nsJSEnvironment.obj] Error 2
Assignee | ||
Comment 6•17 years ago
|
||
This bug, which was about the NSPR and tamarin errors, is fixed. Bug 428529, which is also about actionmonkey being busted on windows, is still a problem.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Summary: Actionmonkey busted on Windows → Actionmonkey busted on Windows with NSPR flags and Tamarin defines
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•