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

Unified Diff: build/android/gyp/write_build_config.py

Issue 1846113002: 🍱 Pass build scripts .interface.jar directly rather than --use-ijars (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove --main-class ref in host_jar.gypi Created 4 years, 9 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 | « build/android/gyp/javac.py ('k') | build/config/android/internal_rules.gni » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/gyp/write_build_config.py
diff --git a/build/android/gyp/write_build_config.py b/build/android/gyp/write_build_config.py
index aad458f05cec64b768afeee280725a47993080c3..8d4cdb1cd71f1977be04bc33f1a13a4d494a5afc 100755
--- a/build/android/gyp/write_build_config.py
+++ b/build/android/gyp/write_build_config.py
@@ -177,6 +177,10 @@ def _FilterUnwantedDepsPaths(dep_paths, target_type):
return ret
+def _AsInterfaceJar(jar_path):
+ return jar_path[:-3] + 'interface.jar'
+
+
def main(argv):
parser = optparse.OptionParser()
build_utils.AddDepfileOption(parser)
@@ -465,15 +469,19 @@ def main(argv):
if options.type in ('java_binary', 'java_library', 'android_apk'):
config['javac']['classpath'] = javac_classpath
+ config['javac']['interface_classpath'] = [
+ _AsInterfaceJar(p) for p in javac_classpath]
config['java'] = {
'full_classpath': java_full_classpath
}
if options.type == 'android_apk':
+ dependency_jars = [c['jar_path'] for c in all_library_deps]
+ all_interface_jars = [
+ _AsInterfaceJar(p) for p in dependency_jars + [options.jar_path]]
config['dist_jar'] = {
- 'dependency_jars': [
- c['jar_path'] for c in all_library_deps
- ]
+ 'dependency_jars': dependency_jars,
+ 'all_interface_jars': all_interface_jars,
}
manifest = AndroidManifest(options.android_manifest)
deps_info['package_name'] = manifest.GetPackageName()
« no previous file with comments | « build/android/gyp/javac.py ('k') | build/config/android/internal_rules.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698