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

Side by Side Diff: build/android/gyp/util/build_device.py

Issue 197693002: [Android] Lint build/android/gyp/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: now using extra_paths_list to avoid F0401s Created 6 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 unified diff | Download patch
« no previous file with comments | « build/android/gyp/strip_library_for_device.py ('k') | build/android/gyp/util/build_utils.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """ A simple device interface for build steps. 5 """ A simple device interface for build steps.
6 6
7 """ 7 """
8 8
9 import logging 9 import logging
10 import os 10 import os
11 import re 11 import re
12 import sys 12 import sys
13 13
14 import build_utils 14 from util import build_utils
15 15
16 BUILD_ANDROID_DIR = os.path.join(os.path.dirname(__file__), '..', '..') 16 BUILD_ANDROID_DIR = os.path.join(os.path.dirname(__file__), '..', '..')
17 sys.path.append(BUILD_ANDROID_DIR) 17 sys.path.append(BUILD_ANDROID_DIR)
18 18
19 from pylib import android_commands 19 from pylib import android_commands
20 20
21 from pylib.android_commands import GetAttachedDevices 21 GetAttachedDevices = android_commands.GetAttachedDevices
22 22
23 23
24 class BuildDevice(object): 24 class BuildDevice(object):
25 def __init__(self, configuration): 25 def __init__(self, configuration):
26 self.id = configuration['id'] 26 self.id = configuration['id']
27 self.description = configuration['description'] 27 self.description = configuration['description']
28 self.install_metadata = configuration['install_metadata'] 28 self.install_metadata = configuration['install_metadata']
29 self.adb = android_commands.AndroidCommands(self.id) 29 self.adb = android_commands.AndroidCommands(self.id)
30 30
31 def RunShellCommand(self, *args, **kwargs): 31 def RunShellCommand(self, *args, **kwargs):
(...skipping 13 matching lines...) Expand all
45 # Matches lines like: 45 # Matches lines like:
46 # -rw-r--r-- system system 7376582 2013-04-19 16:34 \ 46 # -rw-r--r-- system system 7376582 2013-04-19 16:34 \
47 # org.chromium.chrome.shell.apk 47 # org.chromium.chrome.shell.apk
48 # -rw-r--r-- system system 7376582 2013-04-19 16:34 \ 48 # -rw-r--r-- system system 7376582 2013-04-19 16:34 \
49 # org.chromium.chrome.shell-1.apk 49 # org.chromium.chrome.shell-1.apk
50 apk_matcher = lambda s: re.match('.*%s(-[0-9]*)?.apk$' % apk_package, s) 50 apk_matcher = lambda s: re.match('.*%s(-[0-9]*)?.apk$' % apk_package, s)
51 matches = filter(apk_matcher, self.install_metadata) 51 matches = filter(apk_matcher, self.install_metadata)
52 return matches[0] if matches else None 52 return matches[0] if matches else None
53 53
54 54
55 def GetConfigurationForDevice(id): 55 def GetConfigurationForDevice(device_id):
56 adb = android_commands.AndroidCommands(id) 56 adb = android_commands.AndroidCommands(device_id)
57 configuration = None 57 configuration = None
58 has_root = False 58 has_root = False
59 is_online = adb.IsOnline() 59 is_online = adb.IsOnline()
60 if is_online: 60 if is_online:
61 cmd = 'ls -l /data/app; getprop ro.build.description' 61 cmd = 'ls -l /data/app; getprop ro.build.description'
62 cmd_output = adb.RunShellCommand(cmd) 62 cmd_output = adb.RunShellCommand(cmd)
63 has_root = not 'Permission denied' in cmd_output[0] 63 has_root = not 'Permission denied' in cmd_output[0]
64 if not has_root: 64 if not has_root:
65 # Disable warning log messages from EnableAdbRoot() 65 # Disable warning log messages from EnableAdbRoot()
66 logging.getLogger().disabled = True 66 logging.getLogger().disabled = True
67 has_root = adb.EnableAdbRoot() 67 has_root = adb.EnableAdbRoot()
68 logging.getLogger().disabled = False 68 logging.getLogger().disabled = False
69 cmd_output = adb.RunShellCommand(cmd) 69 cmd_output = adb.RunShellCommand(cmd)
70 70
71 configuration = { 71 configuration = {
72 'id': id, 72 'id': device_id,
73 'description': cmd_output[-1], 73 'description': cmd_output[-1],
74 'install_metadata': cmd_output[:-1], 74 'install_metadata': cmd_output[:-1],
75 } 75 }
76 return configuration, is_online, has_root 76 return configuration, is_online, has_root
77 77
78 78
79 def WriteConfigurations(configurations, path): 79 def WriteConfigurations(configurations, path):
80 # Currently we only support installing to the first device. 80 # Currently we only support installing to the first device.
81 build_utils.WriteJson(configurations[:1], path, only_if_changed=True) 81 build_utils.WriteJson(configurations[:1], path, only_if_changed=True)
82 82
83 83
84 def ReadConfigurations(path): 84 def ReadConfigurations(path):
85 return build_utils.ReadJson(path) 85 return build_utils.ReadJson(path)
86 86
87 87
88 def GetBuildDevice(configurations): 88 def GetBuildDevice(configurations):
89 assert len(configurations) == 1 89 assert len(configurations) == 1
90 return BuildDevice(configurations[0]) 90 return BuildDevice(configurations[0])
91 91
92 92
93 def GetBuildDeviceFromPath(path): 93 def GetBuildDeviceFromPath(path):
94 configurations = ReadConfigurations(path) 94 configurations = ReadConfigurations(path)
95 if len(configurations) > 0: 95 if len(configurations) > 0:
96 return GetBuildDevice(ReadConfigurations(path)) 96 return GetBuildDevice(ReadConfigurations(path))
97 return None 97 return None
98 98
OLDNEW
« no previous file with comments | « build/android/gyp/strip_library_for_device.py ('k') | build/android/gyp/util/build_utils.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698