Closed Bug 379518 Opened 13 years ago Closed 13 years ago

sync up with breakpad svn, yet again

Categories

(Toolkit :: Crash Reporting, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: ted, Assigned: ted)

References

Details

Attachments

(4 files, 2 obsolete files)

We need to get campd's change for building with the 10.3.9 SDK, my Linux dump_syms change, and we should probably wait for the Linux dump_syms change to remove the OpenSSL dependency.  There were also some OS X dump_syms changes that make it not broken with our symbols, so we'll need this before we can turn on symbol uploading on the OS X tinderbox.
Attached patch synced to revision 155 (obsolete) — Splinter Review
This is a cvs diff of our current Breakpad snapshot to the latest Breakpad SVN (revision 155).
Attachment #263713 - Flags: review?(mark)
This is just the Makefile.in changes required by the sync up (adding some new source files).  This is part of the previous diff.
Forgot to diff -N.
Attachment #263713 - Attachment is obsolete: true
Attachment #263716 - Flags: review?(mark)
Attachment #263713 - Flags: review?(mark)
Comment on attachment 263716 [details] [diff] [review]
synced to revision 155 (with new files)

Of course this doesn't build on ppc, I filed a breakpad issue on that:
http://code.google.com/p/google-breakpad/issues/detail?id=160
Attachment #263716 - Flags: review?(mark)
Ok, committed that, this is synced to revision 156.  The Makefiles patch above is still a subset of this.
Attachment #263716 - Attachment is obsolete: true
Attachment #263761 - Flags: review?(mark)
Comment on attachment 263761 [details] [diff] [review]
synced to revision 156

You can bring it up to 157 to also pick up your revert.
Attachment #263761 - Flags: review?(mark) → review+
Checked in synced with revision 157.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Need to resync to 159 to pick up my fix for issue 163.  Should be painless.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Synced up to revision 159.  This is just my fix for issue 163 + a newline that got added to another file.
Attachment #264037 - Flags: review?(mark)
Attachment #264037 - Flags: review?(mark) → review+
Checked in.
Status: REOPENED → RESOLVED
Closed: 13 years ago13 years ago
Resolution: --- → FIXED
Had to pick up one more fix.  :-/
Comment on attachment 264150 [details] [diff] [review]
Synced to revision 162

Checked in with r=bsmedberg over IRC.
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.