You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Filip Maj (JIRA)" <ji...@apache.org> on 2017/04/06 18:27:41 UTC

[jira] [Created] (CB-12640) listing targets with cli scripts does not work with v26 of android sdk tools

Filip Maj created CB-12640:
------------------------------

             Summary: listing targets with cli scripts does not work with v26 of android sdk tools
                 Key: CB-12640
                 URL: https://issues.apache.org/jira/browse/CB-12640
             Project: Apache Cordova
          Issue Type: Bug
          Components: Android
    Affects Versions: 6.2.1
            Reporter: Filip Maj
            Assignee: Filip Maj


Another Android SDK Tools release, another new problem! The {{android}} command is back and has completely different behaviour from SDK Tools 25.3.x. It now:
  a) prints out a deprecation warning
  b) does not accept {{list targets}} (must be {{list target}}) as a command
  c) does not print out an error message telling users to use {{avdmanager}} or {{sdkmanager}}

In 25.3.x, {{android}} would exit with a non-zero exit code and print out an error message saying the command is no longer available.

In 25.2.x and below, {{android}} was the go-to command for all things android-related.

The [existing code for listing targets|https://github.com/apache/cordova-android/blob/master/bin/templates/cordova/lib/android_sdk.js] currently follows this flow:
1. Run {{android}} to list targets. If that works, parse targets out and exit.
2. If {{android}} fails with a non-zero exit code and a particular error message (the one that would be output in 25.3.x version of the tools), leverage {{sdkmanager}}, parse its output for installed targets.

[~bowserj] brought up a good point to just use {{avdmanager list target}} instead of {{sdkmanager}} - it maintains the same output format for target listing as the old {{android}} tool does, which should allow us to remove the [code we have in place to convert {{sdkmanager}} output to the old {{android}} output format|https://github.com/apache/cordova-android/blob/master/bin/templates/cordova/lib/android_sdk.js#L107]. 

Another solid [~bowserj] piece of advice is to try to use the {{avdmanager}} tool first before falling back to {{android}}.

I will attempt to implement both suggestions above, and add test cases leveraging output fixtures from the various Android SDK Tool versions.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org