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

Issue 1084002: Fix VS project generation for the crash generation app. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 10 months ago by Cris Neckar
Modified:
10 years, 10 months ago
Reviewers:
Mark Mentovai
CC:
google-breakpad-dev_googlegroups.com
Base URL:
http://google-breakpad.googlecode.com/svn/trunk/src/
Visibility:
Public.

Description

Fix VS project generation for the crash generation app.

BUG=N/A
TEST=N/A
R=mark@chromium.org

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

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M client/windows/tests/crash_generation_app/crash_generation_app.gyp View 1 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 4
Cris Neckar
10 years, 10 months ago #1
Mark Mentovai
LGTM https://breakpad.appspot.com/1084002/diff/1/client/windows/tests/crash_generation_app/crash_generation_app.gyp File client/windows/tests/crash_generation_app/crash_generation_app.gyp (right): https://breakpad.appspot.com/1084002/diff/1/client/windows/tests/crash_generation_app/crash_generation_app.gyp#newcode48 client/windows/tests/crash_generation_app/crash_generation_app.gyp:48: 'libraries': [ Why’s this so far to the ...
10 years, 10 months ago #2
Cris Neckar
https://breakpad.appspot.com/1084002/diff/1/client/windows/tests/crash_generation_app/crash_generation_app.gyp File client/windows/tests/crash_generation_app/crash_generation_app.gyp (right): https://breakpad.appspot.com/1084002/diff/1/client/windows/tests/crash_generation_app/crash_generation_app.gyp#newcode48 client/windows/tests/crash_generation_app/crash_generation_app.gyp:48: 'libraries': [ On 2014/01/17 21:51:44, Mark Mentovai wrote: > ...
10 years, 10 months ago #3
Cris Neckar
10 years, 10 months ago #4
Message was sent while issue was closed.
Committed patchset #2 manually as r1273 (presubmit successful).
Sign in to reply to this message.

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