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

Issue 115664: Explicitly enable /INCREMENTAL linking for gyp-generated Debug builds,... (Closed)

Created:
11 years, 7 months ago by sgk
Modified:
9 years, 5 months ago
Reviewers:
bradn
CC:
chromium-reviews_googlegroups.com, darin (slow to review), Mark Mentovai
Visibility:
Public.

Description

Explicitly enable /INCREMENTAL linking for gyp-generated Debug builds. Explicitly disable it (/INCREMENTAL:NO) for Release, and for the following targets that require it: chrome.dll interactive_ui_tests.exe perf_tests.exe unit_tests.exe Explicitly specificy /SUBSYSTEM:CONSOLE as default for linking, and match current practice by overriding with /SUBSYSTEM:WINDOWS for: chrome.exe chrome.dll media_player.exe sandbox_poc.exe TEST=none BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=16698

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+66 lines, -1 line) Patch
M build/common.gypi View 1 4 chunks +20 lines, -1 line 0 comments Download
M chrome/chrome.gyp View 1 5 chunks +36 lines, -0 lines 0 comments Download
M media/media.gyp View 1 chunk +5 lines, -0 lines 0 comments Download
M sandbox/sandbox.gyp View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
sgk
11 years, 7 months ago (2009-05-21 22:02:25 UTC) #1
bradn
LGTM
11 years, 7 months ago (2009-05-21 22:07:09 UTC) #2
sgk
11 years, 7 months ago (2009-05-21 23:55:40 UTC) #3
Updating with overrides for targets that explicitly need /INCREMENTAL:NO and
/SUBSYSTEM:WINDOWS.  Should match current practice.

mmentovai, would appreciate your eyes on the conditions in case you spot
opportunities for more effective/readable patterns.

Powered by Google App Engine
This is Rietveld 408576698