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

Unified Diff: webrtc/examples/BUILD.gn

Issue 2300413002: GN Templates: Move common_config to the template. (Closed)
Patch Set: This might be it. Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/examples/BUILD.gn
diff --git a/webrtc/examples/BUILD.gn b/webrtc/examples/BUILD.gn
index de88afb1defc9feaff787d3edb21a8ee74fced3f..9e63e2df8bea496017f0ccce4a27e04b7303aaa1 100644
--- a/webrtc/examples/BUILD.gn
+++ b/webrtc/examples/BUILD.gn
@@ -140,7 +140,6 @@ if (is_ios || (is_mac && target_cpu != "x86")) {
"objc/AppRTCDemo/common/ARDUtilities.m",
]
configs += [
- "..:common_config",
":warnings_config",
"//build/config/compiler:enable_arc",
]
@@ -209,7 +208,6 @@ if (is_ios || (is_mac && target_cpu != "x86")) {
"objc/AppRTCDemo/RTCSessionDescription+JSON.m",
]
configs += [
- "..:common_config",
"//build/config/compiler:enable_arc",
":warnings_config",
]
@@ -299,7 +297,6 @@ if (is_ios || (is_mac && target_cpu != "x86")) {
"objc/AppRTCDemo/mac/APPRTCViewController.m",
]
configs += [
- "..:common_config",
"..:common_objc",
"//build/config/compiler:enable_arc",
]
@@ -360,7 +357,6 @@ if (is_ios || (is_mac && target_cpu != "x86")) {
"objc/AppRTCDemo/third_party/SocketRocket/SRWebSocket.m",
]
configs += [
- "..:common_config",
"//build/config/compiler:enable_arc",
":socketrocket_warning_config",
]
@@ -413,6 +409,7 @@ if (is_linux || is_win) {
"peerconnection/client/peer_connection_client.cc",
"peerconnection/client/peer_connection_client.h",
]
+ suppressed_configs += [ rtc_common_config ]
kjellander_webrtc 2016/09/05 06:50:28 Add a TODO(ehmaldonado) here, preferably pointing
if (is_clang) {
# Suppress warnings from the Chromium Clang plugin.
# See http://code.google.com/p/webrtc/issues/detail?id=163 for details.

Powered by Google App Engine
This is Rietveld 408576698