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

Unified Diff: mojo/devtools/common/devtoolslib/paths.py

Issue 1781063002: Only set the ADB path if it exists. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/devtools/common/devtoolslib/paths.py
diff --git a/mojo/devtools/common/devtoolslib/paths.py b/mojo/devtools/common/devtoolslib/paths.py
index 8c592aea2decb4e99cfdebde8baa14fe7ca4706b..870dd0a9af0510b13544f3706e5de8642348a267 100644
--- a/mojo/devtools/common/devtoolslib/paths.py
+++ b/mojo/devtools/common/devtoolslib/paths.py
@@ -59,8 +59,10 @@ def infer_params(is_android, is_debug, target_cpu):
params['build_dir_path'] = build_dir_path
if is_android:
params['shell_path'] = os.path.join(build_dir_path, 'apks', 'MojoShell.apk')
- params['adb_path'] = os.path.join(root_path, 'third_party', 'android_tools',
- 'sdk', 'platform-tools', 'adb')
+ adb_path = os.path.join(root_path, 'third_party', 'android_tools',
+ 'sdk', 'platform-tools', 'adb')
+ if os.path.isfile(adb_path):
+ params['adb_path'] = adb_path
else:
params['shell_path'] = os.path.join(build_dir_path, 'mojo_shell')
« 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