Closed
Bug 551850
Opened 14 years ago
Closed 14 years ago
Electrolysis: xpcshell build fails
Categories
(Core :: IPC, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: lusian, Assigned: lusian)
Details
Attachments
(1 file)
1.06 KB,
patch
|
cjones
:
review+
|
Details | Diff | Splinter Review |
xpcshell.cpp c:\mozilla-build\e10s\obj-i686-pc-mingw32\dist\include\jsobj.h(365) : warning C4800: 'jsval *const ' : forcing value to bool 'true' or 'false' (performance warning) ../../../../dist/include\jscntxt.h(286) : warning C4800: 'JSStackFrame *const ' : forcing value to bool 'true' or 'false' (performance warning) ../../../../dist/include\jscntxt.h(333) : warning C4099: 'js::TraceMonitor' : type name first seen using 'class' now seen using 'struct' c:\mozilla-build\e10s\obj-i686-pc-mingw32\dist\include\jsprvtd.h(150) : see declaration of 'js::TraceMonitor' ../../../../dist/include\jscntxt.h(1396) : warning C4273: 'js_TraceContext' : inconsistent dll linkage c:\mozilla-build\e10s\obj-i686-pc-mingw32\dist\include\jsgc.h(184) : see previous definition of 'js_TraceContext' ../../../../dist/include\jscntxt.h(1263) : warning C4355: 'this' : used in base member initializer list C:\Program Files\Microsoft Visual Studio 9.0\VC\INCLUDE\xlocale(342) : warning C4530: C++ exception handler used, but unwind semantics are not enabled. Specify /EHsc c:/mozilla-build/e10s/ipc/chromium/src\base/basictypes.h(58) : error C2371: 'int8' : redefinition; different basic types c:\mozilla-build\e10s\obj-i686-pc-mingw32\dist\include\jsotypes.h(108) : see declaration of 'int8' make[5]: *** [xpcshell.obj] Error 2
Comment 1•14 years ago
|
||
Jae, Is this still breaking for you? Only on Win XP, I assume?
Assignee | ||
Comment 2•14 years ago
|
||
> Is this still breaking for you? Yes. > Only on Win XP, I assume? I don't have access to other systems, so I don't know if this bug happens only on Win XP.
Assignee | ||
Comment 3•14 years ago
|
||
39352:9d62cf3cabba & 39354:8be7c01b3524 broke the build on my machine.
Attachment #433454 -
Flags: review?(jones.chris.g)
Updated•14 years ago
|
Attachment #433454 -
Flags: review?(jones.chris.g) → review+
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Updated•14 years ago
|
Assignee: nobody → lusian
Comment 4•14 years ago
|
||
http://hg.mozilla.org/projects/electrolysis/rev/9948b8e644ae
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•14 years ago
|
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•