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

Unified Diff: build/android/gradle/generate_gradle.py

Issue 2722973002: Revert of Make generate_gradle.py work with preview SDKs. (Closed)
Patch Set: Created 3 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 | « build/android/gradle/android.jinja ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/gradle/generate_gradle.py
diff --git a/build/android/gradle/generate_gradle.py b/build/android/gradle/generate_gradle.py
index 8cd33e651c1a8fe2f34822411c82c677167c5f18..2044aa375775905240472d0042bb002f3246c282 100755
--- a/build/android/gradle/generate_gradle.py
+++ b/build/android/gradle/generate_gradle.py
@@ -92,9 +92,9 @@
output_file.write(data)
-def _ReadPropertiesFile(path):
- with open(path) as f:
- return dict(l.rstrip().split('=', 1) for l in f if '=' in l)
+def _ReadBuildVars(output_dir):
+ with open(os.path.join(output_dir, 'build_vars.txt')) as f:
+ return dict(l.rstrip().split('=', 1) for l in f)
def _RunNinja(output_dir, args):
@@ -514,11 +514,9 @@
variables['template_type'] = target_type
variables['use_gradle_process_resources'] = (
generator.use_gradle_process_resources)
- source_properties = _ReadPropertiesFile(
- os.path.join(build_vars['android_sdk_build_tools'], 'source.properties'))
- variables['build_tools_version'] = source_properties['Pkg.Revision']
- variables['compile_sdk_version'] = (
- 'android-%s' % build_vars['android_sdk_version'])
+ variables['build_tools_version'] = (
+ build_vars['android_sdk_build_tools_version'])
+ variables['compile_sdk_version'] = build_vars['android_sdk_version']
variables['main'] = generator.Generate(entry)
bootclasspath = gradle.get('bootclasspath')
if bootclasspath:
@@ -662,7 +660,7 @@
_gradle_output_dir = os.path.abspath(
args.project_dir.replace('$CHROMIUM_OUTPUT_DIR', output_dir))
jinja_processor = jinja_template.JinjaProcessor(_FILE_DIR)
- build_vars = _ReadPropertiesFile(os.path.join(output_dir, 'build_vars.txt'))
+ build_vars = _ReadBuildVars(output_dir)
generator = _ProjectContextGenerator(_gradle_output_dir, build_vars,
args.use_gradle_process_resources, jinja_processor, args.split_projects)
logging.warning('Creating project at: %s', generator.project_dir)
« no previous file with comments | « build/android/gradle/android.jinja ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698