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

Unified Diff: talk/libjingle.gyp

Issue 1521993002: Revert of Add APK targets to build libjingle tests for Android. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 years 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 | « talk/app/webrtc/webrtcsdp_unittest.cc ('k') | talk/libjingle_tests.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: talk/libjingle.gyp
diff --git a/talk/libjingle.gyp b/talk/libjingle.gyp
index 158254a3b2fef449225261feddafd9f0c01675df..f1ce18d96a4c309e8933113124f4419e64606c58 100755
--- a/talk/libjingle.gyp
+++ b/talk/libjingle.gyp
@@ -43,8 +43,8 @@
['OS=="linux" or OS=="android"', {
'targets': [
{
- 'target_name': 'libjingle_peerconnection_jni',
- 'type': 'static_library',
+ 'target_name': 'libjingle_peerconnection_so',
+ 'type': 'shared_library',
'dependencies': [
'<(webrtc_root)/system_wrappers/system_wrappers.gyp:field_trial_default',
'libjingle_peerconnection',
@@ -61,55 +61,6 @@
'include_dirs': [
'<(libyuv_dir)/include',
],
- 'conditions': [
- ['OS=="linux"', {
- 'include_dirs': [
- '<(java_home)/include',
- '<(java_home)/include/linux',
- ],
- }],
- ['build_json==1', {
- 'dependencies': [
- '<(DEPTH)/third_party/jsoncpp/jsoncpp.gyp:jsoncpp',
- ],
- 'export_dependent_settings': [
- '<(DEPTH)/third_party/jsoncpp/jsoncpp.gyp:jsoncpp',
- ],
- }],
- ['OS=="android"', {
- 'sources': [
- 'app/webrtc/androidvideocapturer.cc',
- 'app/webrtc/androidvideocapturer.h',
- 'app/webrtc/java/jni/androidmediacodeccommon.h',
- 'app/webrtc/java/jni/androidmediadecoder_jni.cc',
- 'app/webrtc/java/jni/androidmediadecoder_jni.h',
- 'app/webrtc/java/jni/androidmediaencoder_jni.cc',
- 'app/webrtc/java/jni/androidmediaencoder_jni.h',
- 'app/webrtc/java/jni/androidnetworkmonitor_jni.cc',
- 'app/webrtc/java/jni/androidnetworkmonitor_jni.h',
- 'app/webrtc/java/jni/androidvideocapturer_jni.cc',
- 'app/webrtc/java/jni/androidvideocapturer_jni.h',
- 'app/webrtc/java/jni/surfacetexturehelper_jni.cc',
- 'app/webrtc/java/jni/surfacetexturehelper_jni.h',
- ]
- }],
- ],
- },
- {
- 'target_name': 'libjingle_peerconnection_so',
- 'type': 'shared_library',
- 'dependencies': [
- 'libjingle_peerconnection',
- 'libjingle_peerconnection_jni',
- ],
- 'sources': [
- 'app/webrtc/java/jni/jni_onload.cc',
- ],
- 'variables': {
- # This library uses native JNI exports; tell GYP so that the
- # required symbols will be kept.
- 'use_native_jni_exports': 1,
- },
'conditions': [
['OS=="linux"', {
'defines': [
@@ -129,6 +80,30 @@
},
}],
],
+ }],
+ ['OS=="android"', {
+ 'sources': [
+ 'app/webrtc/java/jni/androidvideocapturer_jni.cc',
+ 'app/webrtc/java/jni/androidvideocapturer_jni.h',
+ ],
+ 'variables': {
+ # This library uses native JNI exports; tell GYP so that the
+ # required symbols will be kept.
+ 'use_native_jni_exports': 1,
+ },
+ }],
+ ['OS=="android" and build_with_chromium==0', {
+ 'sources': [
+ 'app/webrtc/java/jni/androidmediacodeccommon.h',
+ 'app/webrtc/java/jni/androidmediadecoder_jni.cc',
+ 'app/webrtc/java/jni/androidmediadecoder_jni.h',
+ 'app/webrtc/java/jni/androidmediaencoder_jni.cc',
+ 'app/webrtc/java/jni/androidmediaencoder_jni.h',
+ 'app/webrtc/java/jni/androidnetworkmonitor_jni.cc',
+ 'app/webrtc/java/jni/androidnetworkmonitor_jni.h',
+ 'app/webrtc/java/jni/surfacetexturehelper_jni.cc',
+ 'app/webrtc/java/jni/surfacetexturehelper_jni.h',
+ ]
}],
],
},
@@ -819,6 +794,14 @@
'app/webrtc/webrtcsessiondescriptionfactory.cc',
'app/webrtc/webrtcsessiondescriptionfactory.h',
],
+ 'conditions': [
+ ['OS=="android" and build_with_chromium==0', {
+ 'sources': [
+ 'app/webrtc/androidvideocapturer.h',
+ 'app/webrtc/androidvideocapturer.cc',
+ ],
+ }],
+ ],
}, # target libjingle_peerconnection
],
}
« no previous file with comments | « talk/app/webrtc/webrtcsdp_unittest.cc ('k') | talk/libjingle_tests.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698