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

Side by Side Diff: chrome/chrome_installer.gypi

Issue 9701050: Make setup.exe compatible with the component build. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: comment nits Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'variables': { 6 'variables': {
7 'lastchange_path': '<(DEPTH)/build/util/LASTCHANGE', 7 'lastchange_path': '<(DEPTH)/build/util/LASTCHANGE',
8 # 'branding_dir' is set in the 'conditions' section at the bottom. 8 # 'branding_dir' is set in the 'conditions' section at the bottom.
9 }, 9 },
10 'conditions': [ 10 'conditions': [
(...skipping 306 matching lines...) Expand 10 before | Expand all | Expand 10 after
317 '--output_dir=<(PRODUCT_DIR)', 317 '--output_dir=<(PRODUCT_DIR)',
318 '--input_file=<(RULE_INPUT_PATH)', 318 '--input_file=<(RULE_INPUT_PATH)',
319 '--header_output_dir=<(INTERMEDIATE_DIR)', 319 '--header_output_dir=<(INTERMEDIATE_DIR)',
320 # TODO(sgk): may just use environment variables 320 # TODO(sgk): may just use environment variables
321 #'--distribution=$(CHROMIUM_BUILD)', 321 #'--distribution=$(CHROMIUM_BUILD)',
322 '--distribution=_google_chrome', 322 '--distribution=_google_chrome',
323 ], 323 ],
324 }, 324 },
325 ], 325 ],
326 'conditions': [ 326 'conditions': [
327 ['component == "shared_library"', {
328 'copies': [
329 {
330 'destination': '<(PRODUCT_DIR)',
331 'files': [
332 'installer/setup/setup.exe.manifest',
333 ],
334 },
335 ],
336 'msvs_settings': {
337 'VCLinkerTool': {
338 'GenerateManifest': 'false',
339 },
340 },
341 }],
327 # TODO(mark): <(branding_dir) should be defined by the 342 # TODO(mark): <(branding_dir) should be defined by the
328 # global condition block at the bottom of the file, but 343 # global condition block at the bottom of the file, but
329 # this doesn't work due to the following issue: 344 # this doesn't work due to the following issue:
330 # 345 #
331 # http://code.google.com/p/gyp/issues/detail?id=22 346 # http://code.google.com/p/gyp/issues/detail?id=22
332 # 347 #
333 # Remove this block once the above issue is fixed. 348 # Remove this block once the above issue is fixed.
334 [ 'branding == "Chrome"', { 349 [ 'branding == "Chrome"', {
335 'variables': { 350 'variables': {
336 'branding_dir': 'app/theme/google_chrome', 351 'branding_dir': 'app/theme/google_chrome',
(...skipping 637 matching lines...) Expand 10 before | Expand all | Expand 10 after
974 'variables': { 989 'variables': {
975 'branding_dir': 'app/theme/google_chrome', 990 'branding_dir': 'app/theme/google_chrome',
976 }, 991 },
977 }, { # else branding!="Chrome" 992 }, { # else branding!="Chrome"
978 'variables': { 993 'variables': {
979 'branding_dir': 'app/theme/chromium', 994 'branding_dir': 'app/theme/chromium',
980 }, 995 },
981 }], 996 }],
982 ], 997 ],
983 } 998 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698