[ 
https://issues.apache.org/jira/browse/CB-12640?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15994034#comment-15994034
 ] 

ASF subversion and git services commented on CB-12640:
------------------------------------------------------

Commit 46e47a3c5cc0c7f36538e7a092fd4d079aa1996d in cordova-android's branch 
refs/heads/6.2.x from filmaj
[ https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;h=46e47a3 ]

CB-12640: support for android sdk tools 26.0.1. simplified target parsing by 
using avdmanager instead of sdkmanager. flipped target parsing logic so that it 
always tries to use avdmanager to retrieve targets first, then falls back to 
android command if avdmanager cannot be found (and errors with ENOENT). updated 
tests.


> 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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to