Note: There are a few cases of duplicates in user autocompletion which are being worked on.

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

RESOLVED FIXED in mozilla22

Status

()

Core
Gecko Profiler
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: rkent, Assigned: rkent)

Tracking

unspecified
mozilla22
x86_64
Windows 7
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
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

4 years ago
Created attachment 727313 [details] [diff] [review]
Add StandardInteger.h

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.
(Assignee)

Updated

4 years ago
Blocks: 779291
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

4 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.
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

4 years ago
Created attachment 729783 [details] [diff] [review]
This also works

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/integration/mozilla-inbound/rev/1562dc5d65c8
Flags: in-testsuite-
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/1562dc5d65c8
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.