Uploaded image for project: 'Apache Cordova'
  1. Apache Cordova
  2. CB-12640

listing targets with cli scripts does not work with v26 of android sdk tools

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 6.2.1
    • None
    • cordova-android
    • None

    Description

      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 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.

      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.

      Attachments

        Issue Links

          Activity

            People

              filmaj Filip Maj
              filmaj Filip Maj
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: