nsExceptionHandler.cpp fails to build for Android/x86

RESOLVED FIXED

Status

()

Toolkit
Crash Reporting
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: glandium, Unassigned)

Tracking

unspecified
x86
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(blocking-fennec1.0 -, fennec+)

Details

(Reporter)

Description

6 years ago
In file included from /home/mh/mozilla-central/toolkit/crashreporter/nsExceptionHandler.cpp:82:
/home/mh/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/linux/handler/exception_handler.h:201: error: field 'context' has incomplete type
/home/mh/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/linux/handler/exception_handler.h:204: error: field 'float_state' has incomplete type
In file included from /home/mh/mozilla-central/toolkit/crashreporter/nsExceptionHandler.cpp:83:
/home/mh/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/linux/minidump_writer/linux_dumper.h:48: error: invalid use of incomplete type 'struct google_breakpad::user'
/home/mh/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/linux/minidump_writer/linux_dumper.h:48: error: forward declaration of 'struct google_breakpad::user'
/home/mh/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/linux/minidump_writer/linux_dumper.h:48: error: invalid type in declaration before ';' token
/home/mh/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/linux/minidump_writer/linux_dumper.h:91: error: 'user_regs_struct' does not name a type
/home/mh/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/linux/minidump_writer/linux_dumper.h:92: error: 'user_fpregs_struct' does not name a type
/home/mh/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/linux/minidump_writer/linux_dumper.h:96: error: 'user_fpxregs_struct' does not name a type
/home/mh/mozilla-central/toolkit/crashreporter/nsExceptionHandler.cpp: In function 'void CrashReporter::FileIDToGUID(const char*, u_int8_t*)':
/home/mh/mozilla-central/toolkit/crashreporter/nsExceptionHandler.cpp:298: warning: comparison between signed and unsigned integer expressions
x86 is not blocking 1.0
blocking-fennec1.0: --- → +
blocking-fennec1.0: + → -
This is probably some Breakpad code I wrote assuming that Android == arm.
Component: General → Breakpad Integration
Product: Fennec Native → Toolkit
QA Contact: general → breakpad.integration
tracking-fennec: --- → +

Comment 3

6 years ago
Is this fixed ? I just built android/x86 with the release mozconfig, and it doesn't have breakpad disabled.
I believe Ted updated breakpad on September 25th, which fixed this. See bug #791775
There were some upstream Breakpad fixes that fixed the Android/x86 case. Thanks for testing, I forgot about this bug because I never set a dependency.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Depends on: 791775
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.