public/cordova-android
public/cordova-android
9
0
Fork 0
forked from github/cordova-android
Code Pull Requests Activity

CB-12524: Forgot to check for the android command on the target check

Browse Source
This commit is contained in:
Joe Bowser 2017-03-13 11:16:48 -07:00
parent 2d3f3cae13
commit 9dbffa6ee8
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
bin/lib/check_reqs.js
View File

@ -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) {