aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorandroid-build-team Robot <android-build-team-robot@google.com>2017-10-20 07:13:45 +0000
committerandroid-build-team Robot <android-build-team-robot@google.com>2017-10-20 07:13:45 +0000
commitb83aafdedf02b3ea0b5109457f674cbce3b670ba (patch)
tree94234f9ba8eab6ccb8e804352f3dc7b648dc8ea5
parentf7580f0d094683dc042d77cc4c014343efce7ce7 (diff)
parent61763e713e66ccd122d1631b3ab4a4f9a201116a (diff)
downloadtradefederation-b83aafdedf02b3ea0b5109457f674cbce3b670ba.tar.gz
Change-Id: Ie2540672b4e2b53510457e7d4949bde8fe85f479
-rw-r--r--src/com/android/tradefed/util/AaptParser.java20
1 files changed, 15 insertions, 5 deletions
diff --git a/src/com/android/tradefed/util/AaptParser.java b/src/com/android/tradefed/util/AaptParser.java
index a69dd7ec3..58a8b39b2 100644
--- a/src/com/android/tradefed/util/AaptParser.java
+++ b/src/com/android/tradefed/util/AaptParser.java
@@ -103,21 +103,31 @@ public class AaptParser {
* @return the {@link AaptParser} or <code>null</code> if failed to extract the information
*/
public static AaptParser parse(File apkFile) {
- CommandResult result = RunUtil.getDefault().runTimedCmd(AAPT_TIMEOUT_MS,
- "aapt", "dump", "badging", apkFile.getAbsolutePath());
+ CommandResult result =
+ RunUtil.getDefault()
+ .runTimedCmdRetry(
+ AAPT_TIMEOUT_MS,
+ 0L,
+ 2,
+ "aapt",
+ "dump",
+ "badging",
+ apkFile.getAbsolutePath());
String stderr = result.getStderr();
- if (stderr != null && stderr.length() > 0) {
+ if (stderr != null && !stderr.isEmpty()) {
CLog.e("aapt dump badging stderr: %s", stderr);
}
- if (result.getStatus() == CommandStatus.SUCCESS) {
+ if (CommandStatus.SUCCESS.equals(result.getStatus())) {
AaptParser p = new AaptParser();
if (p.parse(result.getStdout()))
return p;
return null;
}
- CLog.e("Failed to run aapt on %s", apkFile.getAbsoluteFile());
+ CLog.e(
+ "Failed to run aapt on %s. stdout: %s",
+ apkFile.getAbsoluteFile(), result.getStdout());
return null;
}