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

Unified Diff: build/android/devil/devil_env.py

Issue 1539403002: [Android] Revise devil environment handling of host dependencies. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix empty config w/o output_directory Created 5 years 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/devil/devil_dependencies.json ('k') | build/android/devil/devil_env_test.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/devil/devil_env.py
diff --git a/build/android/devil/devil_env.py b/build/android/devil/devil_env.py
index 8249e53c0aa6745735f87d36c2173610bb88c630..07bb658ced30a444a4af5d0694b7982edc798046 100644
--- a/build/android/devil/devil_env.py
+++ b/build/android/devil/devil_env.py
@@ -6,6 +6,7 @@ import contextlib
import json
import logging
import os
+import platform
import sys
import tempfile
import threading
@@ -37,11 +38,11 @@ _DEVIL_DEFAULT_CONFIG = os.path.abspath(os.path.join(
_LEGACY_ENVIRONMENT_VARIABLES = {
'ADB_PATH': {
'dependency_name': 'adb',
- 'platform': 'android_linux2',
+ 'platform': 'linux_x86_64',
},
'ANDROID_SDK_ROOT': {
'dependency_name': 'android_sdk',
- 'platform': 'android_linux2',
+ 'platform': 'linux_x86_64',
},
}
@@ -136,9 +137,9 @@ class _Environment(object):
def GetPlatform(arch=None, device=None):
- if not arch:
- arch = device.product_cpu_abi if device else sys.platform
- return 'android_%s' % arch
+ if device:
+ return 'android_%s' % (arch or device.product_cpu_abi)
+ return 'linux_%s' % platform.machine()
config = _Environment()
« no previous file with comments | « build/android/devil/devil_dependencies.json ('k') | build/android/devil/devil_env_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698