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

Unified Diff: net/net.gyp

Issue 10913083: Remove {base,net}_java dependencies from GYP client targets on Android. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address Peter's comment Created 8 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
« content/content.gyp ('K') | « media/media.gyp ('k') | sql/sql.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/net.gyp
diff --git a/net/net.gyp b/net/net.gyp
index 6b00c158094ee16621cea54e5991791ec8a19d07..4eb98fe37c6bf05e7563dcd5f7e9076c7c46dd9f 100644
--- a/net/net.gyp
+++ b/net/net.gyp
@@ -1100,6 +1100,14 @@
],
},
],
+ ['OS=="android" and _toolset=="target"', {
+ 'dependencies': [
+ 'net_java',
+ ],
+ 'export_dependent_settings': [
+ 'net_java',
+ ],
+ }],
[ 'OS == "android"', {
'dependencies': [
'../third_party/openssl/openssl.gyp:openssl',
@@ -2078,10 +2086,10 @@
'java_in_dir': '../net/android/java',
},
'dependencies': [
- '../base/base.gyp:base_java',
+ '../base/base.gyp:base',
],
'export_dependent_settings': [
- '../base/base.gyp:base_java',
+ '../base/base.gyp:base',
],
'includes': [ '../build/java.gypi' ],
},
@@ -2093,12 +2101,14 @@
'java_in_dir': '../net/android/javatests',
},
'dependencies': [
- '../base/base.gyp:base_java',
+ '../base/base.gyp:base',
'../base/base.gyp:base_java_test_support',
+ 'net_java',
],
'export_dependent_settings': [
- '../base/base.gyp:base_java',
+ '../base/base.gyp:base',
'../base/base.gyp:base_java_test_support',
+ 'net_java',
],
'includes': [ '../build/java.gypi' ],
},
« content/content.gyp ('K') | « media/media.gyp ('k') | sql/sql.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698