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

Unified Diff: webrtc/media/media.gyp

Issue 1693803002: Make it possible to exclude device management code from rtc_media target. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Flattended conditions hierarchy, added Android and iOS and updated comment Created 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/build/common.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/media/media.gyp
diff --git a/webrtc/media/media.gyp b/webrtc/media/media.gyp
index e4960e16c9b48f5a8a2b4cbcd212cba60baf00c2..2a5295734a056bc0acd333c39e94b34d5790e49f 100644
--- a/webrtc/media/media.gyp
+++ b/webrtc/media/media.gyp
@@ -77,9 +77,6 @@
'base/yuvframegenerator.cc',
'base/yuvframegenerator.h',
'devices/deviceinfo.h',
- 'devices/devicemanager.cc',
- 'devices/devicemanager.h',
- 'devices/dummydevicemanager.h',
'devices/videorendererfactory.h',
'engine/nullwebrtcvideoengine.h',
'engine/simulcast.cc',
@@ -124,6 +121,17 @@
4389, # signed/unsigned mismatch.
],
'conditions': [
+ ['include_internal_device_management==1', {
+ 'sources': [
+ 'devices/devicemanager.cc',
+ 'devices/devicemanager.h',
+ ],
+ }, {
+ 'sources': [
+ 'devices/dummydevicemanager.cc',
+ 'devices/dummydevicemanager.h',
+ ],
+ }],
['build_libyuv==1', {
'dependencies': ['<(DEPTH)/third_party/libyuv/libyuv.gyp:libyuv',],
}],
@@ -158,7 +166,7 @@
'<(webrtc_root)/modules/modules.gyp:video_render_module_internal_impl',
],
}],
- ['OS=="linux"', {
+ ['OS=="linux" and include_internal_device_management==1', {
'sources': [
'devices/libudevsymboltable.cc',
'devices/libudevsymboltable.h',
@@ -168,17 +176,6 @@
'devices/v4llookup.cc',
'devices/v4llookup.h',
],
- 'conditions': [
- ['use_gtk==1', {
- 'sources': [
- 'devices/gtkvideorenderer.cc',
- 'devices/gtkvideorenderer.h',
- ],
- 'cflags': [
- '<!@(pkg-config --cflags gobject-2.0 gthread-2.0 gtk+-2.0)',
- ],
- }],
- ],
'include_dirs': [
'third_party/libudev'
],
@@ -186,13 +183,19 @@
'-lrt',
],
}],
+ ['OS=="linux" and use_gtk==1', {
+ 'sources': [
+ 'devices/gtkvideorenderer.cc',
+ 'devices/gtkvideorenderer.h',
+ ],
+ 'cflags': [
+ '<!@(pkg-config --cflags gobject-2.0 gthread-2.0 gtk+-2.0)',
+ ],
+ }],
['OS=="win"', {
'sources': [
'devices/gdivideorenderer.cc',
'devices/gdivideorenderer.h',
- 'devices/win32deviceinfo.cc',
- 'devices/win32devicemanager.cc',
- 'devices/win32devicemanager.h',
],
'msvs_settings': {
'VCLibrarianTool': {
@@ -200,36 +203,34 @@
'd3d9.lib',
'gdi32.lib',
'strmiids.lib',
+ ],
+ },
+ },
+ }],
+ ['OS=="win" and include_internal_device_management==1', {
+ 'sources': [
+ 'devices/win32deviceinfo.cc',
+ 'devices/win32devicemanager.cc',
+ 'devices/win32devicemanager.h',
+ ],
+ 'msvs_settings': {
+ 'VCLibrarianTool': {
+ 'AdditionalDependencies': [
'winmm.lib',
],
},
},
}],
- ['OS=="mac"', {
+ ['OS=="mac" and include_internal_device_management==1', {
'sources': [
'devices/macdeviceinfo.cc',
'devices/macdevicemanager.cc',
'devices/macdevicemanager.h',
'devices/macdevicemanagermm.mm',
],
- 'conditions': [
- ['target_arch=="ia32"', {
- 'sources': [
- 'devices/carbonvideorenderer.cc',
- 'devices/carbonvideorenderer.h',
- ],
- 'link_settings': {
- 'xcode_settings': {
- 'OTHER_LDFLAGS': [
- '-framework Carbon',
- ],
- },
- },
- }],
- ],
'xcode_settings': {
'WARNING_CFLAGS': [
- # TODO(ronghuawu): Update macdevicemanager.cc to stop using
+ # TODO(perkj): Update macdevicemanager.cc to stop using
# deprecated functions and remove this flag.
'-Wno-deprecated-declarations',
],
@@ -251,7 +252,20 @@
},
},
}],
- ['OS=="ios"', {
+ ['OS=="mac" and target_arch=="ia32"', {
+ 'sources': [
+ 'devices/carbonvideorenderer.cc',
+ 'devices/carbonvideorenderer.h',
+ ],
+ 'link_settings': {
+ 'xcode_settings': {
+ 'OTHER_LDFLAGS': [
+ '-framework Carbon',
+ ],
+ },
+ },
+ }],
+ ['OS=="ios" and include_internal_device_management==1', {
'sources': [
'devices/mobiledevicemanager.cc',
],
@@ -276,7 +290,7 @@
'CARBON_DEPRECATED=YES',
],
}],
- ['OS=="android"', {
+ ['OS=="android" and include_internal_device_management==1', {
'sources': [
'devices/mobiledevicemanager.cc',
],
« no previous file with comments | « webrtc/build/common.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698