Skip to content

Made p4a call sdkmanager instead of android in new Android SDK versions #1051

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

Closed
wants to merge 1 commit into from

Conversation

inclement
Copy link
Member

@inclement inclement commented May 14, 2017

No description provided.

@inclement inclement added this to the 0.5 milestone May 14, 2017
@nocarryr
Copy link
Contributor

In working through the same errors as in #1050 I've found that my android-sdk layout places avdmanager under ./tools/bin/avdmanager instead of ./bin/avdmanager (as shown in L232-L233).

I also have a depreciated version of the ./tools/android callable, so in testing I had to move the avdmanager check to the top of the stack.

Not sure if this is what everyone is experiencing or not, but this is on a freshly installed VM (16.04) and fresh download of the sdk

@inclement
Copy link
Member Author

@nocarryr Yeah, this PR is not quite right as I downloaded the SDK from scratch and misunderstood the directory structure. I have an improved version that isn't pushed yet.

@nocarryr
Copy link
Contributor

@inclement 👍 Thanks, just thought I'd share the discovery. Have you also noticed any issues with a non-existent build.xml expected under ./tools/ant/ (within the sdk root)?

This may be a separate issue, but I thought I'd ask here in case you've come across it

@inclement
Copy link
Member Author

@nocarryr Thanks for reporting it, it's always useful!

The build.xml problem is because the build tools no longer ship with ant, it is also deprecated. I think you can install an older version for now, or tito has done the work to make p4a use gradle instead (PR at #1042), which we're working to review and merge.

@nocarryr
Copy link
Contributor

Thanks for the info, I haven't checked in on p4a progress in awhile so I'm​ catching up. Great work btw

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

Successfully merging this pull request may close these issues.

2 participants