Closed
Bug 853097
Opened 12 years ago
Closed 12 years ago
cfi_frame-info.cc won't compile with VS2008SP1/VC9 after bug 779291
Categories
(Core :: Gecko Profiler, defect)
Tracking
()
RESOLVED
FIXED
mozilla22
People
(Reporter: rkent, Assigned: rkent)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
When I do a clobber build of Thunderbird, I'm getting a failure in mozilla core code that looks like this: c:\tb\1-central\src\mozilla\config\rules.mk:1056:0$ c:/tb/1-central/tb-pydebug/mozilla/_virtualenv/Scripts/python.exe -O c:/tb/1-central/src/mozilla/build/cl.py cl -Focfi_frame_info.obj -c -D_HAS_EXCEPTIONS=0 -I../../../../../dist/stl_wrapp ers -DMOZ_THUNDERBIRD=1 -DNO_NSPR_10_SUPPORT -Ic:/tb/1-central/src/mozilla/toolkit/crashreporter/google-breakpad/src/pr ocessor/../.. -Ic:/tb/1-central/src/mozilla/toolkit/crashreporter/google-breakpad/src/processor/.. -Ic:/tb/1-central/src /mozilla/toolkit/crashreporter/google-breakpad/src/processor -I. -I../../../../../dist/include -Ic:/tb/1-central/tb-pyd ebug/mozilla/dist/include/nspr -Ic:/tb/1-central/tb-pydebug/mozilla/dist/include/nss -wd4099 -TP -nologo -W3 -Gy -Fdgenerated.pdb -wd4251 -wd4345 -wd4351 -wd4482 -wd4800 -wd4819 -we4553 -GR- -DDEBUG -D_DEBUG -DTRACING -Zi -Oy- -MDd -FI ../../../../../dist/include/mozilla-config.h -DMOZILLA_CLIENT c:/tb/1-central/src/mozilla/toolkit/crashre porter/google-breakpad/src/processor/cfi_frame_info.cc cfi_frame_info.cc c:\tb\1-central\src\mozilla\toolkit\crashreporter\google-breakpad\src\processor/postfix_evaluator-inl.h(319) : error C20 65: 'int64_t' : undeclared identifier There are additional errors associated with other types.
Assignee | ||
Comment 1•12 years ago
|
||
This patch allows the compile to continue. I'm not asking for a review though because I don't really understand why this is needed, so I must be missing something.
Comment 2•12 years ago
|
||
This was supposed to be fixed by: http://mxr.mozilla.org/mozilla-central/source/configure.in#6031 http://mxr.mozilla.org/mozilla-central/source/toolkit/crashreporter/google-breakpad/src/google_breakpad/common/breakpad_types.h Are you building with --disable-crashreporter?
Assignee | ||
Comment 3•12 years ago
|
||
"Are you building with --disable-crashreporter?" Yes I believe that I am - because of some previous compile issue that is probably long gone.
Comment 4•12 years ago
|
||
Ok, we probably just need to move this AC_DEFINE_UNQUOTED: http://mxr.mozilla.org/mozilla-central/source/configure.in#6041 outside of the 'if test -n "$MOZ_CRASHREPORTER";' block. Can you test that? You can just move it down to immediately after that whole if block.
Assignee | ||
Comment 5•12 years ago
|
||
This patch, which is my interpretation of comment 4, also allows me to successfully compile.
Attachment #727313 -
Attachment is obsolete: true
Comment 6•12 years ago
|
||
Comment on attachment 729783 [details] [diff] [review] This also works Looks fine.
Attachment #729783 -
Flags: review+
Updated•12 years ago
|
Assignee: nobody → kent
Keywords: checkin-needed
Comment 7•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1562dc5d65c8
Flags: in-testsuite-
Keywords: checkin-needed
Comment 8•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1562dc5d65c8
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in
before you can comment on or make changes to this bug.
Description
•