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

Use sdkmanager for Android builds #837

Closed
wants to merge 2 commits into from
Closed

Use sdkmanager for Android builds #837

wants to merge 2 commits into from

Conversation

Jawnnypoo
Copy link

@Jawnnypoo Jawnnypoo commented Mar 2, 2017

First of all, let me say I have no experience setting up a cookbook.

I am trying to get the ball rolling on updating Android builds to use sdkmanager instead of the now somewhat obsolete update command.

Main reasoning for this is that certain newer features in the SDK are only available to download via the sdkmanager command line tool, such as CMake for Android builds (see this issue)

The tests are passing on my repo, but I have doubts that I have set this up correctly. For reference, here is how I use sdkmanager on builds done on GitLab CI

https://gitlab.com/Commit451/LabCoat/blob/master/.gitlab-ci.yml

Is there someone who can help me out on this? Possibly @gildegoma ?

@gildegoma
Copy link
Contributor

@Jawnnypoo thanks for bringing this. This issue has also been reported upstream (gildegoma/chef-android-sdk#34).
For an "urgent patch" review, I'd recommend to get advices from @rjaros87 and @Brantone.

@rjaros87
Copy link

rjaros87 commented Mar 2, 2017

Looks good. But what with the backward compatibility.

Look at rjaros87/chef-android-sdk#2

@Jawnnypoo
Copy link
Author

Ahh @rjaros87 I did not realize this was a WIP in the other repo. I will go ahead and close this one, as it seems that one is more complete. I am guessing that will eventually make it into the official cookbook?

@Jawnnypoo Jawnnypoo closed this Mar 2, 2017
@gildegoma
Copy link
Contributor

@Jawnnypoo 👍 Yup, I'll take care to encourage Travis builders to update travis-cookbooks repo as soon as the upstream fix is released.

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.

3 participants