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

Unified Diff: build/android/incremental_install.py

Issue 1326813003: Tweak logging in incremental_install.py to know when each part is done (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@device-more-caching
Patch Set: Created 5 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/incremental_install.py
diff --git a/build/android/incremental_install.py b/build/android/incremental_install.py
index 9020607adff84c62a76a93348aa9e23b25e4aefd..7e1316ac8ea1ef6a4a9813e1ae661612855cdf86 100755
--- a/build/android/incremental_install.py
+++ b/build/android/incremental_install.py
@@ -92,6 +92,7 @@ def main():
allow_cached_props=True)
else:
device.Install(args.apk_path, reinstall=True)
+ logging.info('Finished installing .apk')
jbudorick 2015/09/02 18:14:19 I know this feeling. I'd recommend using -v/--verb
agrieve 2015/09/02 18:34:04 Was planning on adding this, but does seem like a
# Push .so files to the device (if they have changed).
def do_push_libs():
@@ -99,6 +100,8 @@ def main():
device_lib_dir = posixpath.join(device_incremental_dir, 'lib')
device.PushChangedFiles([(args.lib_dir, device_lib_dir)],
delete_device_stale=True)
+ logging.info('Finished pushing native libs')
+
# Concurrency here speeds things up quite a bit, but DeviceUtils hasn't
# been designed for multi-threading. Enabling only because this is a
# developer-only tool.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698