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

[android] Make build work with instructions in docs #698

Merged
merged 1 commit into from
Oct 15, 2021
Merged

Conversation

shroff
Copy link
Collaborator

@shroff shroff commented Sep 26, 2021

build.gradle currently does not respect the MAPBOX_DOWNLOADS_TOKEN
project property as subbested by the official docs for Android at
https://docs.mapbox.com/android/maps/guides/install/.

This approach does not interfere with the SDK_REGISTRY_TOKEN
environment variable used for CI, but just uses the official approach as
a fallback.

Also updated docs

`build.gradle` currently does not respect the `MAPBOX_DOWNLOADS_TOKEN`
project property as subbested by the official docs for Android at
https://docs.mapbox.com/android/maps/guides/install/.

This approach does not interfere with the `SDK_REGISTRY_TOKEN`
environment variable used for CI, but just uses the official approach as
a fallback.

Also updated docs
@shroff shroff requested review from n8han and tobrun September 26, 2021 06:22
@shroff shroff had a problem deploying to ANDROID_CI_DOWNLOADS_TOKEN September 26, 2021 06:22 Failure
@shroff shroff had a problem deploying to ANDROID_CI_DOWNLOADS_TOKEN September 26, 2021 06:22 Failure
Copy link
Collaborator

@n8han n8han left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@felix-ht
Copy link
Collaborator

@shroff if you remove tobrun from the reviewers the ci should be able to run and we can merge it

@shroff shroff removed the request for review from tobrun October 15, 2021 16:01
@shroff
Copy link
Collaborator Author

shroff commented Oct 15, 2021

@felix-ht I'm not sure why that would help, but I did it anyway

@shroff shroff merged commit c1a6919 into master Oct 15, 2021
@felix-ht
Copy link
Collaborator

@shroff because some parts of the pipeline only run if all the users marked as reviewers did the review. As tobrun didn't do it the build steps did not run.

@philiplindberg
Copy link
Contributor

It seems like the build.gradle project property was actually named incorrectly. According to the Mapbox Android docs, it should be named MAPBOX_DOWNLOADS_TOKEN, but it looks like it's called MAPBOX_DOWNLOAD_TOKEN (DOWNLOAD without the 'S') in gradle.build.

@shroff shroff deleted the registry-token branch October 24, 2021 17:10
@felix-ht
Copy link
Collaborator

@philiplindberg can you create a merge request for that?

@philiplindberg
Copy link
Contributor

@felix-ht looks like it's already merged: #721

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.

4 participants