From 9dbffa6ee83ee0019cebb6262c78958e7e3b1086 Mon Sep 17 00:00:00 2001 From: Joe Bowser Date: Mon, 13 Mar 2017 11:16:48 -0700 Subject: [PATCH] CB-12524: Forgot to check for the android command on the target check --- bin/lib/check_reqs.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/bin/lib/check_reqs.js b/bin/lib/check_reqs.js index c6898e75..eb1e4d15 100644 --- a/bin/lib/check_reqs.js +++ b/bin/lib/check_reqs.js @@ -257,6 +257,8 @@ module.exports.check_android = function() { module.exports.getAbsoluteAndroidCmd = function () { var cmd = forgivingWhichSync('android'); + if(cmd.length === 0) + cmd = forgivingWhichSync('avdmanager'); if (process.platform === 'win32') { return '"' + cmd + '"'; } @@ -270,8 +272,11 @@ module.exports.check_android_target = function(originalError) { // Google Inc.:Google APIs:20 // Google Inc.:Glass Development Kit Preview:20 var valid_target = module.exports.get_target(); - var msg = 'Android SDK not found. Make sure that it is installed. If it is not at the default location, set the ANDROID_HOME environment variable.'; - return tryCommand('android list targets --compact', msg) + // Changing "targets" to "target" is stupid and makes more code. Thanks Google! + var cmd = "android list targets --compact"; + if(forgivingWhichSync('avdmanager').length > 0) + cmd = "avdmanager list target --compact"; + return tryCommand(cmd, msg) .then(function(output) { var targets = output.split('\n'); if (targets.indexOf(valid_target) >= 0) {