Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(43)

Issue 79873002: [VS Addin] Set the 'NonCoreWin' property to 'true' (Closed)

Created:
7 years, 1 month ago by Sam Clegg
Modified:
7 years, 1 month ago
Reviewers:
binji
CC:
native-client-reviews_googlegroups.com
Visibility:
Public.

Description

[VS Addin] Set the 'NonCoreWin' property to 'true' Without this the MS build files for 2012 were adding the core windows libraries to the list of dependencies. This fixes the 2012 tests which are currently failing on the bots. R=binji@chromium.org Committed: https://code.google.com/p/nativeclient-sdk/source/detail?r=1568

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M visual_studio/NativeClientVSAddIn/InstallerResources/NaCl64/Microsoft.Cpp.NaCl64.props View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Sam Clegg
7 years, 1 month ago (2013-11-21 00:21:58 UTC) #1
binji
lgtm
7 years, 1 month ago (2013-11-21 00:23:24 UTC) #2
Sam Clegg
7 years, 1 month ago (2013-11-21 00:26:57 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r1568 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698