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

Side by Side Diff: webrtc/tools/BUILD.gn

Issue 2408133008: GN: Exclude suppressions of Chromium Clang warnings for Chromium builds. (Closed)
Patch Set: Created 4 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 | « webrtc/test/fuzzers/BUILD.gn ('k') | webrtc/video/BUILD.gn » ('j') | 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) 2014 The WebRTC project authors. All Rights Reserved. 1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license 3 # Use of this source code is governed by a BSD-style license
4 # that can be found in the LICENSE file in the root of the source 4 # that can be found in the LICENSE file in the root of the source
5 # tree. An additional intellectual property rights grant can be found 5 # tree. An additional intellectual property rights grant can be found
6 # in the file PATENTS. All contributing project authors may 6 # in the file PATENTS. All contributing project authors may
7 # be found in the AUTHORS file in the root of the source tree. 7 # be found in the AUTHORS file in the root of the source tree.
8 8
9 import("//third_party/protobuf/proto_library.gni") 9 import("//third_party/protobuf/proto_library.gni")
10 import("../build/webrtc.gni") 10 import("../build/webrtc.gni")
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 87
88 rtc_static_library("frame_editing_lib") { 88 rtc_static_library("frame_editing_lib") {
89 sources = [ 89 sources = [
90 "frame_editing/frame_editing_lib.cc", 90 "frame_editing/frame_editing_lib.cc",
91 "frame_editing/frame_editing_lib.h", 91 "frame_editing/frame_editing_lib.h",
92 ] 92 ]
93 93
94 # TODO(jschuh): Bug 1348: fix this warning. 94 # TODO(jschuh): Bug 1348: fix this warning.
95 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ] 95 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
96 96
97 if (is_clang) { 97 if (!build_with_chromium && is_clang) {
98 # Suppress warnings from the Chromium Clang plugin. 98 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
99 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
100 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 99 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
101 } 100 }
102 101
103 deps = [ 102 deps = [
104 "../common_video", 103 "../common_video",
105 ] 104 ]
106 } 105 }
107 106
108 rtc_executable("frame_editor") { 107 rtc_executable("frame_editor") {
109 sources = [ 108 sources = [
110 "frame_editing/frame_editing.cc", 109 "frame_editing/frame_editing.cc",
111 ] 110 ]
112 111
113 deps = [ 112 deps = [
114 ":command_line_parser", 113 ":command_line_parser",
115 ":frame_editing_lib", 114 ":frame_editing_lib",
116 "//build/win:default_exe_manifest", 115 "//build/win:default_exe_manifest",
117 ] 116 ]
118 } 117 }
119 118
120 # It doesn't make sense to build this tool without the ADM enabled. 119 # It doesn't make sense to build this tool without the ADM enabled.
121 if (rtc_include_internal_audio_device) { 120 if (rtc_include_internal_audio_device) {
122 rtc_executable("force_mic_volume_max") { 121 rtc_executable("force_mic_volume_max") {
123 sources = [ 122 sources = [
124 "force_mic_volume_max/force_mic_volume_max.cc", 123 "force_mic_volume_max/force_mic_volume_max.cc",
125 ] 124 ]
126 125
127 if (is_clang) { 126 if (!build_with_chromium && is_clang) {
128 # Suppress warnings from the Chromium Clang plugin. 127 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
129 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
130 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 128 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
131 } 129 }
132 130
133 deps = [ 131 deps = [
134 "../modules/audio_device:audio_device", 132 "../modules/audio_device:audio_device",
135 "../system_wrappers:system_wrappers_default", 133 "../system_wrappers:system_wrappers_default",
136 "//build/win:default_exe_manifest", 134 "//build/win:default_exe_manifest",
137 ] 135 ]
138 } 136 }
139 } 137 }
(...skipping 10 matching lines...) Expand all
150 sources = [ 148 sources = [
151 "event_log_visualizer/analyzer.cc", 149 "event_log_visualizer/analyzer.cc",
152 "event_log_visualizer/analyzer.h", 150 "event_log_visualizer/analyzer.h",
153 "event_log_visualizer/plot_base.cc", 151 "event_log_visualizer/plot_base.cc",
154 "event_log_visualizer/plot_base.h", 152 "event_log_visualizer/plot_base.h",
155 "event_log_visualizer/plot_protobuf.cc", 153 "event_log_visualizer/plot_protobuf.cc",
156 "event_log_visualizer/plot_protobuf.h", 154 "event_log_visualizer/plot_protobuf.h",
157 "event_log_visualizer/plot_python.cc", 155 "event_log_visualizer/plot_python.cc",
158 "event_log_visualizer/plot_python.h", 156 "event_log_visualizer/plot_python.h",
159 ] 157 ]
160 if (is_clang && !is_nacl) { 158 if (!build_with_chromium && is_clang) {
161 # Suppress warnings from the Chromium Clang plugin. 159 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
162 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
163 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 160 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
164 } 161 }
165 defines = [ "ENABLE_RTC_EVENT_LOG" ] 162 defines = [ "ENABLE_RTC_EVENT_LOG" ]
166 deps = [ 163 deps = [
167 "../logging:rtc_event_log_impl", 164 "../logging:rtc_event_log_impl",
168 "../logging:rtc_event_log_parser", 165 "../logging:rtc_event_log_parser",
169 "../modules/congestion_controller:congestion_controller", 166 "../modules/congestion_controller:congestion_controller",
170 "../modules/rtp_rtcp:rtp_rtcp", 167 "../modules/rtp_rtcp:rtp_rtcp",
171 "../system_wrappers:system_wrappers_default", 168 "../system_wrappers:system_wrappers_default",
172 "//build/config/sanitizers:deps", 169 "//build/config/sanitizers:deps",
173 ] 170 ]
174 public_deps = [ 171 public_deps = [
175 ":chart_proto", 172 ":chart_proto",
176 "../logging:rtc_event_log_parser", 173 "../logging:rtc_event_log_parser",
177 ] 174 ]
178 } 175 }
179 } 176 }
180 177
181 # Exclude tools depending on gflags since that's not available in Chromium. 178 # Exclude tools depending on gflags since that's not available in Chromium.
182 if (rtc_include_tests) { 179 if (rtc_include_tests) {
183 if (rtc_enable_protobuf) { 180 if (rtc_enable_protobuf) {
184 rtc_executable("event_log_visualizer") { 181 rtc_executable("event_log_visualizer") {
185 testonly = true 182 testonly = true
186 sources = [ 183 sources = [
187 "event_log_visualizer/main.cc", 184 "event_log_visualizer/main.cc",
188 ] 185 ]
189 186
190 if (is_clang && !is_nacl) { 187 if (!build_with_chromium && is_clang) {
191 # Suppress warnings from the Chromium Clang plugin. 188 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163) .
192 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
193 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 189 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
194 } 190 }
195 191
196 defines = [ "ENABLE_RTC_EVENT_LOG" ] 192 defines = [ "ENABLE_RTC_EVENT_LOG" ]
197 deps = [ 193 deps = [
198 ":event_log_visualizer_utils", 194 ":event_log_visualizer_utils",
199 "//third_party/gflags", 195 "//third_party/gflags",
200 ] 196 ]
201 } 197 }
202 } 198 }
203 199
204 rtc_executable("activity_metric") { 200 rtc_executable("activity_metric") {
205 testonly = true 201 testonly = true
206 sources = [ 202 sources = [
207 "agc/activity_metric.cc", 203 "agc/activity_metric.cc",
208 ] 204 ]
209 205
210 if (is_clang) { 206 if (!build_with_chromium && is_clang) {
211 # Suppress warnings from the Chromium Clang plugin. 207 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
212 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
213 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 208 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
214 } 209 }
215 210
216 deps = [ 211 deps = [
217 "../modules/audio_processing", 212 "../modules/audio_processing",
218 "../system_wrappers:metrics_default", 213 "../system_wrappers:metrics_default",
219 "//build/win:default_exe_manifest", 214 "//build/win:default_exe_manifest",
220 "//testing/gtest", 215 "//testing/gtest",
221 "//third_party/gflags", 216 "//third_party/gflags",
222 ] 217 ]
(...skipping 16 matching lines...) Expand all
239 234
240 sources = [ 235 sources = [
241 "frame_analyzer/video_quality_analysis_unittest.cc", 236 "frame_analyzer/video_quality_analysis_unittest.cc",
242 "frame_editing/frame_editing_unittest.cc", 237 "frame_editing/frame_editing_unittest.cc",
243 "simple_command_line_parser_unittest.cc", 238 "simple_command_line_parser_unittest.cc",
244 ] 239 ]
245 240
246 # TODO(jschuh): Bug 1348: fix this warning. 241 # TODO(jschuh): Bug 1348: fix this warning.
247 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ] 242 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
248 243
249 if (is_clang) { 244 if (!build_with_chromium && is_clang) {
250 # Suppress warnings from the Chromium Clang plugin. 245 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
251 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
252 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 246 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
253 } 247 }
254 248
255 deps = [ 249 deps = [
256 ":command_line_parser", 250 ":command_line_parser",
257 ":frame_editing_lib", 251 ":frame_editing_lib",
258 ":video_quality_analysis", 252 ":video_quality_analysis",
259 "../test:test_support_main", 253 "../test:test_support_main",
260 "//testing/gtest", 254 "//testing/gtest",
261 ] 255 ]
(...skipping 18 matching lines...) Expand all
280 ] 274 ]
281 outputs = [ 275 outputs = [
282 "$root_build_dir/{{source_file_part}}", 276 "$root_build_dir/{{source_file_part}}",
283 ] 277 ]
284 deps = [ 278 deps = [
285 "../logging:rtc_event_log_proto", 279 "../logging:rtc_event_log_proto",
286 ] 280 ]
287 } 281 }
288 } 282 }
289 } 283 }
OLDNEW
« no previous file with comments | « webrtc/test/fuzzers/BUILD.gn ('k') | webrtc/video/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698