Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warning to fix: android.bat shouldn't be recommended anymore #741

Closed
Songbird0 opened this issue Dec 1, 2018 · 2 comments
Closed

Warning to fix: android.bat shouldn't be recommended anymore #741

Songbird0 opened this issue Dec 1, 2018 · 2 comments

Comments

@Songbird0
Copy link

Hi,

When cordova requirements command is typed and any Android target hasn't been assigned, cordova recommends to use android.bat to install a fresh Android target. However, sdkmanager.bat should be used instead of android.bat.

I would be very happy to fix it myself, but I don't know the location of this warning.

@Songbird0 Songbird0 changed the title Warning to fix: android.bat shouldn't be recommanded anymore Warning to fix: android.bat shouldn't be recommended anymore Dec 1, 2018
@janpio
Copy link
Member

janpio commented Dec 1, 2018

Highly probable it's in https://github.com/apache/cordova-android - using the search there it should be easy enough to find.

Songbird0 referenced this issue in Songbird0/cordova-android Dec 2, 2018
Hi,

Like said:

```
**************************************************************************
The "android" command is deprecated.
For manual SDK, AVD, and project management, please use Android Studio.
For command-line tools, use tools\bin\sdkmanager.bat
and tools\bin\avdmanager.bat
**************************************************************************
```

The conditionnal has been removed to use systematically `sdkmanager` instead of `android.bat`.

See [this issue](https://github.com/apache/cordova/issues/61).
@janpio janpio transferred this issue from apache/cordova Jun 7, 2019
@erisu erisu closed this as completed Jul 29, 2021
@erisu
Copy link
Member

erisu commented Jul 29, 2021

Closing this issue as PR #1083 mentions that it closes this ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants