Closed Bug 853097 Opened 11 years ago Closed 11 years ago

cfi_frame-info.cc won't compile with VS2008SP1/VC9 after bug 779291

Categories

(Core :: Gecko Profiler, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: rkent, Assigned: rkent)

References

Details

Attachments

(1 file, 1 obsolete file)

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.
Attached patch Add StandardInteger.h (obsolete) — Splinter Review
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.
Blocks: 779291
"Are you building with --disable-crashreporter?"

Yes I believe that I am - because of some previous compile issue that is probably long gone.
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.
Attached patch This also worksSplinter Review
This patch, which is my interpretation of comment 4, also allows me to successfully compile.
Attachment #727313 - Attachment is obsolete: true
Comment on attachment 729783 [details] [diff] [review]
This also works

Looks fine.
Attachment #729783 - Flags: review+
Assignee: nobody → kent
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/1562dc5d65c8
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: