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

Side by Side Diff: chrome/chrome_exe.gypi

Issue 9307019: Cleanup: Merge some OS specific sections in chrome/*.gypi. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 10 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
« no previous file with comments | « chrome/chrome_browser.gypi ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 'targets': [ 6 'targets': [
7 { 7 {
8 'target_name': 'chrome', 8 'target_name': 'chrome',
9 'type': 'executable', 9 'type': 'executable',
10 'mac_bundle': 1, 10 'mac_bundle': 1,
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 ], 155 ],
156 }], 156 }],
157 ], 157 ],
158 'sources': [ 158 'sources': [
159 'app/chrome_dll_resource.h', 159 'app/chrome_dll_resource.h',
160 'app/chrome_main.cc', 160 'app/chrome_main.cc',
161 'app/chrome_main_delegate.cc', 161 'app/chrome_main_delegate.cc',
162 'app/chrome_main_delegate.h', 162 'app/chrome_main_delegate.h',
163 ], 163 ],
164 }], 164 }],
165 ['OS=="linux"', {
166 'conditions': [
167 ['branding=="Chrome"', {
168 'dependencies': [
169 'linux_installer_configs',
170 ],
171 }],
172 ['selinux==0', {
173 'dependencies': [
174 '../sandbox/sandbox.gyp:sandbox',
175 ],
176 }],
177 # For now, do not build nacl_helper when disable_nacl=1
178 # or when arm is enabled
179 # http://code.google.com/p/gyp/issues/detail?id=239
180 ['disable_nacl==0 and target_arch!="arm"', {
181 'dependencies': [
182 '../native_client/src/trusted/service_runtime/linux/nacl_bootstr ap.gyp:nacl_helper_bootstrap',
183 'nacl_helper',
184 ],
185 }],
186 ],
187 }],
165 ['OS=="mac"', { 188 ['OS=="mac"', {
166 # 'branding' is a variable defined in common.gypi 189 # 'branding' is a variable defined in common.gypi
167 # (e.g. "Chromium", "Chrome") 190 # (e.g. "Chromium", "Chrome")
168 'conditions': [ 191 'conditions': [
169 ['branding=="Chrome"', { 192 ['branding=="Chrome"', {
170 'mac_bundle_resources': [ 193 'mac_bundle_resources': [
171 'app/theme/google_chrome/app.icns', 194 'app/theme/google_chrome/app.icns',
172 'app/theme/google_chrome/document.icns', 195 'app/theme/google_chrome/document.icns',
173 'browser/ui/cocoa/applescript/scripting.sdef', 196 'browser/ui/cocoa/applescript/scripting.sdef',
174 ], 197 ],
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
234 # CFBundleIdentifier of CHROMIUM_BUNDLE_ID 257 # CFBundleIdentifier of CHROMIUM_BUNDLE_ID
235 # CFBundleName of CHROMIUM_SHORT_NAME 258 # CFBundleName of CHROMIUM_SHORT_NAME
236 # CFBundleSignature of CHROMIUM_CREATOR 259 # CFBundleSignature of CHROMIUM_CREATOR
237 # Xcode then replaces these values with the branded values we set 260 # Xcode then replaces these values with the branded values we set
238 # as settings on the target. 261 # as settings on the target.
239 'CHROMIUM_BUNDLE_ID': '<(mac_bundle_id)', 262 'CHROMIUM_BUNDLE_ID': '<(mac_bundle_id)',
240 'CHROMIUM_CREATOR': '<(mac_creator)', 263 'CHROMIUM_CREATOR': '<(mac_creator)',
241 'CHROMIUM_SHORT_NAME': '<(branding)', 264 'CHROMIUM_SHORT_NAME': '<(branding)',
242 }, 265 },
243 'dependencies': [ 266 'dependencies': [
267 # On Mac, make sure we've built chrome_dll, which contains all of
268 # the library code with Chromium functionality.
269 'chrome_dll',
244 'helper_app', 270 'helper_app',
245 'infoplist_strings_tool', 271 'infoplist_strings_tool',
246 'interpose_dependency_shim', 272 'interpose_dependency_shim',
247 'chrome_manifest_bundle', 273 'chrome_manifest_bundle',
248 ], 274 ],
249 'mac_bundle_resources': [ 275 'mac_bundle_resources': [
250 '<(PRODUCT_DIR)/<(mac_bundle_id).manifest', 276 '<(PRODUCT_DIR)/<(mac_bundle_id).manifest',
251 ], 277 ],
252 'actions': [ 278 'actions': [
253 { 279 {
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
374 }, 400 },
375 { 401 {
376 # Make sure there isn't any Objective-C in the browser app's 402 # Make sure there isn't any Objective-C in the browser app's
377 # executable. 403 # executable.
378 'postbuild_name': 'Verify No Objective-C', 404 'postbuild_name': 'Verify No Objective-C',
379 'action': [ 405 'action': [
380 'tools/build/mac/verify_no_objc.sh', 406 'tools/build/mac/verify_no_objc.sh',
381 ], 407 ],
382 }, 408 },
383 ], # postbuilds 409 ], # postbuilds
384 }], 410 }, { # OS != "mac"
385 ['OS=="linux"', {
386 'conditions': [
387 ['branding=="Chrome"', {
388 'dependencies': [
389 'linux_installer_configs',
390 ],
391 }],
392 ['selinux==0', {
393 'dependencies': [
394 '../sandbox/sandbox.gyp:sandbox',
395 ],
396 }],
397 ],
398 }],
399 ['OS != "mac"', {
400 'conditions': [ 411 'conditions': [
401 # TODO: add a: 412 # TODO: add a:
402 # 'product_name': 'chromium' 413 # 'product_name': 'chromium'
403 # whenever we convert the rest of the infrastructure 414 # whenever we convert the rest of the infrastructure
404 # (buildbots etc.) to understand the branding gyp define. 415 # (buildbots etc.) to understand the branding gyp define.
405 # NOTE: chrome/app/theme/chromium/BRANDING and 416 # NOTE: chrome/app/theme/chromium/BRANDING and
406 # chrome/app/theme/google_chrome/BRANDING have the short name 417 # chrome/app/theme/google_chrome/BRANDING have the short name
407 # "chrome" etc.; should we try to extract from there instead? 418 # "chrome" etc.; should we try to extract from there instead?
408 419
409 # On Mac, this is done in chrome_dll.gypi. 420 # On Mac, this is done in chrome_dll.gypi.
410 ['internal_pdf', { 421 ['internal_pdf', {
411 'dependencies': [ 422 'dependencies': [
412 '../pdf/pdf.gyp:pdf', 423 '../pdf/pdf.gyp:pdf',
413 ], 424 ],
414 }], 425 }],
415 ], 426 ],
416 'dependencies': [ 427 'dependencies': [
417 'chrome_resources.gyp:packed_extra_resources', 428 'chrome_resources.gyp:packed_extra_resources',
418 'chrome_resources.gyp:packed_resources', 429 'chrome_resources.gyp:packed_resources',
419 # Copy Flash Player files to PRODUCT_DIR if applicable. Let the .gyp 430 # Copy Flash Player files to PRODUCT_DIR if applicable. Let the .gyp
420 # file decide what to do on a per-OS basis; on Mac, internal plugins 431 # file decide what to do on a per-OS basis; on Mac, internal plugins
421 # go inside the framework, so this dependency is in chrome_dll.gypi. 432 # go inside the framework, so this dependency is in chrome_dll.gypi.
422 '../third_party/adobe/flash/flash_player.gyp:flash_player', 433 '../third_party/adobe/flash/flash_player.gyp:flash_player',
423 ], 434 ],
424 }], 435 }],
425 ['OS=="linux"', {
426 'conditions': [
427 # For now, do not build nacl_helper when disable_nacl=1
428 # or when arm is enabled
429 # http://code.google.com/p/gyp/issues/detail?id=239
430 ['disable_nacl==0 and target_arch!="arm"', {
431 'dependencies': [
432 '../native_client/src/trusted/service_runtime/linux/nacl_bootstr ap.gyp:nacl_helper_bootstrap',
433 'nacl_helper',
434 ],
435 }],
436 ],
437 }],
438 ['OS=="mac"', {
439 'dependencies': [
440 # On Mac, make sure we've built chrome_dll, which contains all of
441 # the library code with Chromium functionality.
442 'chrome_dll',
443 ],
444 }],
445 ['OS=="mac" and asan==1', { 436 ['OS=="mac" and asan==1', {
446 'xcode_settings': { 437 'xcode_settings': {
447 # Override the outer definition of CHROMIUM_STRIP_SAVE_FILE. 438 # Override the outer definition of CHROMIUM_STRIP_SAVE_FILE.
448 'CHROMIUM_STRIP_SAVE_FILE': 'app/app_asan.saves', 439 'CHROMIUM_STRIP_SAVE_FILE': 'app/app_asan.saves',
449 }, 440 },
450 }], 441 }],
451 ['OS=="win"', { 442 ['OS=="win"', {
452 'dependencies': [ 443 'dependencies': [
453 'chrome_dll', 444 'chrome_dll',
454 'chrome_version_resources', 445 'chrome_version_resources',
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
557 'configurations': { 548 'configurations': {
558 'Common_Base': { 549 'Common_Base': {
559 'msvs_target_platform': 'x64', 550 'msvs_target_platform': 'x64',
560 }, 551 },
561 }, 552 },
562 }, 553 },
563 ], 554 ],
564 }], 555 }],
565 ], 556 ],
566 } 557 }
OLDNEW
« no previous file with comments | « chrome/chrome_browser.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698