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

Side by Side Diff: build/common.gypi

Issue 644683004: android_webview: force clang for host targets. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
« no previous file with comments | « no previous file | 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 # IMPORTANT: 5 # IMPORTANT:
6 # Please don't directly include this file if you are building via gyp_chromium, 6 # Please don't directly include this file if you are building via gyp_chromium,
7 # since gyp_chromium is automatically forcing its inclusion. 7 # since gyp_chromium is automatically forcing its inclusion.
8 { 8 {
9 # Variables expected to be overriden on the GYP command line (-D) or by 9 # Variables expected to be overriden on the GYP command line (-D) or by
10 # ~/.gyp/include.gypi. 10 # ~/.gyp/include.gypi.
(...skipping 5561 matching lines...) Expand 10 before | Expand all | Expand 10 after
5572 ], 5572 ],
5573 }, 5573 },
5574 }], 5574 }],
5575 ['gcc_version>=48 and clang==0 and host_clang==1', { 5575 ['gcc_version>=48 and clang==0 and host_clang==1', {
5576 'target_defaults': { 5576 'target_defaults': {
5577 'target_conditions': [ 5577 'target_conditions': [
5578 ['_toolset=="host"', { 'cflags!': [ '-Wno-unused-local-typedefs' ]}], 5578 ['_toolset=="host"', { 'cflags!': [ '-Wno-unused-local-typedefs' ]}],
5579 ], 5579 ],
5580 }, 5580 },
5581 }], 5581 }],
5582 # In the android webview build, force host targets to be compiled with clang
5583 # as the hermetic host gcc is very old on some platforms. This is already
5584 # the default on the current development version of AOSP but we force it
5585 # here in case we need to compile against an older release version.
5586 ['android_webview_build==1', {
Primiano Tucci (use gerrit) 2014/10/09 14:18:12 Shouldn't this be android_webview_build==1 and hos
5587 'target_defaults': {
5588 'target_conditions': [
5589 ['_toolset=="host"', {
5590 'aosp_build_settings': {
5591 'LOCAL_CLANG': 'true',
5592 },
5593 }],
5594 ],
5595 },
5596 }],
5582 # We need a special case to handle the android webview build on mac because 5597 # We need a special case to handle the android webview build on mac because
5583 # the host gcc there doesn't accept this flag, but the target gcc may 5598 # the host gcc there doesn't accept this flag, but the target gcc may
5584 # require it. 5599 # require it.
5585 ['gcc_version>=48 and android_webview_build==1 and host_os=="mac"', { 5600 ['gcc_version>=48 and android_webview_build==1 and host_os=="mac"', {
5586 'target_defaults': { 5601 'target_defaults': {
5587 'target_conditions': [ 5602 'target_conditions': [
5588 ['_toolset=="host"', { 5603 ['_toolset=="host"', {
5589 'cflags!': [ 5604 'cflags!': [
5590 '-Wno-unused-local-typedefs', 5605 '-Wno-unused-local-typedefs',
5591 ], 5606 ],
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
5711 # settings in target dicts. SYMROOT is a special case, because many other 5726 # settings in target dicts. SYMROOT is a special case, because many other
5712 # Xcode variables depend on it, including variables such as 5727 # Xcode variables depend on it, including variables such as
5713 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something 5728 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something
5714 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the 5729 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
5715 # files to appear (when present) in the UI as actual files and not red 5730 # files to appear (when present) in the UI as actual files and not red
5716 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR, 5731 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
5717 # and therefore SYMROOT, needs to be set at the project level. 5732 # and therefore SYMROOT, needs to be set at the project level.
5718 'SYMROOT': '<(DEPTH)/xcodebuild', 5733 'SYMROOT': '<(DEPTH)/xcodebuild',
5719 }, 5734 },
5720 } 5735 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698