Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(240)

Issue 584002: Check in new Windows binaries. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 6 months ago by chrisha
Modified:
11 years, 6 months ago
Reviewers:
Siggi, Mark Mentovai
Base URL:
http://google-breakpad.googlecode.com/svn/trunk/src/
Visibility:
Public.

Description

Check in new Windows binaries.

These binaries include recent changes to pdb_source_line_writer for handling
PDBs with OMAP data. They were built using the GYP build files, MSVS 2010 and
DIA SDK 8.0.

Committed: https://code.google.com/p/google-breakpad/source/detail?r=1177

Patch Set 1 #

Patch Set 2 : Now linked against msdia80.dll. #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M src/tools/windows/binaries/dump_syms.exe View 1 0 chunks +-1 lines, --1 lines 0 comments Download
M src/tools/windows/binaries/symupload.exe View 1 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 11
chrisha
PTAL.
11 years, 6 months ago #1
Siggi
lgtm, for what it's worth. You've verified that these binaries work against msdia80.dll?
11 years, 6 months ago #2
chrisha
This is build against the newest SDK, so uses msdia100.dll. Are we limited to using ...
11 years, 6 months ago #3
Mark Mentovai
Not sure what the bots have on them, but if it’s possible to do this ...
11 years, 6 months ago #4
chrisha
I'm looking into it. Seeing if anybody has an old VS2005 install lying around, or ...
11 years, 6 months ago #5
Siggi
Hey, just be thankful it's not COBOL code :). Upgrading sounds like the sane thing ...
11 years, 6 months ago #6
Mark Mentovai
The mailing list I referred you to before should at least be able to tell ...
11 years, 6 months ago #7
chrisha
Okay, I finally tracked down the 8.0 DIA SDK, and rebuilt these binaries against it. ...
11 years, 6 months ago #8
Siggi
lgtm
11 years, 6 months ago #9
Mark Mentovai
LGTM
11 years, 6 months ago #10
chrisha
11 years, 6 months ago #11
Thanks, committing.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1004:630ec63f810e-tainted