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

Unified Diff: webrtc/webrtc.gyp

Issue 2060873002: Remove webrtc_all target (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Added *.gyp to OWNERS Created 4 years, 6 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
« no previous file with comments | « all.gyp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/webrtc.gyp
diff --git a/webrtc/webrtc.gyp b/webrtc/webrtc.gyp
index e1df4336f135da72a408ddae693899750f87b4b2..8a56fe765c44c196c025fc971560039653e67bec 100644
--- a/webrtc/webrtc.gyp
+++ b/webrtc/webrtc.gyp
@@ -6,101 +6,6 @@
# in the file PATENTS. All contributing project authors may
# be found in the AUTHORS file in the root of the source tree.
{
- 'variables': {
- 'webrtc_all_dependencies': [
- 'base/base.gyp:*',
- 'common.gyp:*',
- 'common_audio/common_audio.gyp:*',
- 'common_video/common_video.gyp:*',
- 'media/media.gyp:*',
- 'modules/modules.gyp:*',
- 'p2p/p2p.gyp:*',
- 'pc/pc.gyp:*',
- 'system_wrappers/system_wrappers.gyp:*',
- 'tools/tools.gyp:*',
- 'voice_engine/voice_engine.gyp:*',
- '<(webrtc_vp8_dir)/vp8.gyp:*',
- '<(webrtc_vp9_dir)/vp9.gyp:*',
- ],
- },
- 'conditions': [
- ['build_with_chromium==0', {
- # TODO(kjellander): Move this to webrtc_all_dependencies once all of talk/
- # has been moved to webrtc/. It can't be processed by Chromium since the
- # reference to buid/java.gypi is using an absolute path (and includes
- # entries cannot contain variables).
- 'variables': {
- 'webrtc_all_dependencies': [
- 'api/api.gyp:*',
- ],
- },
- }],
- ['build_with_chromium==0 and'
- '(OS=="ios" or (OS=="mac" and mac_deployment_target=="10.7"))', {
- # TODO(kjellander): Move this to webrtc_all_dependencies once all of talk/
- # has been moved to webrtc/. It can't be processed by Chromium since the
- # reference to buid/java.gypi is using an absolute path (and includes
- # entries cannot contain variables).
- 'variables': {
- 'webrtc_all_dependencies': [
- 'sdk/sdk.gyp:*',
- ],
- },
- }],
- ['include_tests==1', {
- 'includes': [
- 'webrtc_tests.gypi',
- ],
- }],
- ['enable_protobuf==1', {
- 'targets': [
- {
- # This target should only be built if enable_protobuf is defined
- 'target_name': 'rtc_event_log_proto',
- 'type': 'static_library',
- 'sources': ['call/rtc_event_log.proto',],
- 'variables': {
- 'proto_in_dir': 'call',
- 'proto_out_dir': 'webrtc/call',
- },
- 'includes': ['build/protoc.gypi'],
- },
- ],
- }],
- ['enable_protobuf==1', {
- 'targets': [
- {
- 'target_name': 'rtc_event_log_parser',
- 'type': 'static_library',
- 'sources': [
- 'call/rtc_event_log_parser.cc',
- 'call/rtc_event_log_parser.h',
- ],
- 'dependencies': [
- 'rtc_event_log_proto',
- ],
- 'export_dependent_settings': [
- 'rtc_event_log_proto',
- ],
- },
- ],
- }],
- ['include_tests==1 and enable_protobuf==1', {
- 'targets': [
- {
- 'target_name': 'rtc_event_log2rtp_dump',
- 'type': 'executable',
- 'sources': ['call/rtc_event_log2rtp_dump.cc',],
- 'dependencies': [
- '<(DEPTH)/third_party/gflags/gflags.gyp:gflags',
- 'rtc_event_log_parser',
- 'rtc_event_log_proto',
- 'test/test.gyp:rtp_test_utils'
- ],
- },
- ],
- }],
- ],
'includes': [
'build/common.gypi',
'audio/webrtc_audio.gypi',
@@ -109,27 +14,6 @@
],
'targets': [
{
- 'target_name': 'webrtc_all',
- 'type': 'none',
- 'dependencies': [
- '<@(webrtc_all_dependencies)',
- 'webrtc',
- ],
- 'conditions': [
- ['include_tests==1', {
- 'dependencies': [
- 'api/api_tests.gyp:*',
- 'common_video/common_video_unittests.gyp:*',
- 'rtc_unittests',
- 'system_wrappers/system_wrappers_tests.gyp:*',
- 'test/metrics.gyp:*',
- 'test/test.gyp:*',
- 'webrtc_tests',
- ],
- }],
- ],
- },
- {
'target_name': 'webrtc',
'type': 'static_library',
'sources': [
@@ -185,6 +69,56 @@
}],
],
},
-
- ],
+ ], # targets
+ 'conditions': [
+ ['include_tests==1', {
+ 'includes': [
+ 'webrtc_tests.gypi',
+ ],
+ }],
+ ['enable_protobuf==1', {
+ 'targets': [
+ {
+ # This target should only be built if enable_protobuf is defined
+ 'target_name': 'rtc_event_log_proto',
+ 'type': 'static_library',
+ 'sources': ['call/rtc_event_log.proto',],
+ 'variables': {
+ 'proto_in_dir': 'call',
+ 'proto_out_dir': 'webrtc/call',
+ },
+ 'includes': ['build/protoc.gypi'],
+ },
+ {
+ 'target_name': 'rtc_event_log_parser',
+ 'type': 'static_library',
+ 'sources': [
+ 'call/rtc_event_log_parser.cc',
+ 'call/rtc_event_log_parser.h',
+ ],
+ 'dependencies': [
+ 'rtc_event_log_proto',
+ ],
+ 'export_dependent_settings': [
+ 'rtc_event_log_proto',
+ ],
+ },
+ ],
+ }],
+ ['include_tests==1 and enable_protobuf==1', {
+ 'targets': [
+ {
+ 'target_name': 'rtc_event_log2rtp_dump',
+ 'type': 'executable',
+ 'sources': ['call/rtc_event_log2rtp_dump.cc',],
+ 'dependencies': [
+ '<(DEPTH)/third_party/gflags/gflags.gyp:gflags',
+ 'rtc_event_log_parser',
+ 'rtc_event_log_proto',
+ 'test/test.gyp:rtp_test_utils'
+ ],
+ },
+ ],
+ }],
+ ], # conditions
}
« no previous file with comments | « all.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698