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 embedding fixes - migrate to maven #852

Merged
merged 3 commits into from
Jan 13, 2022
Merged

Conversation

felix-ht
Copy link
Collaborator

@felix-ht felix-ht commented Jan 13, 2022

Based on #817 - moved to a local branch to enable full ci builds. Thanks @lfofelipe for doing the maven changes.

Also did some changes to the android embedding that have been missed - this caused some build failures.

@felix-ht felix-ht temporarily deployed to ANDROID_CI_DOWNLOADS_TOKEN January 13, 2022 10:45 Inactive
@felix-ht felix-ht temporarily deployed to ANDROID_CI_DOWNLOADS_TOKEN January 13, 2022 10:45 Inactive
@felix-ht felix-ht requested a review from tobrun January 13, 2022 10:46
@felix-ht felix-ht requested a review from m0nac0 January 13, 2022 11:10
@tobrun tobrun merged commit bd2ade0 into master Jan 13, 2022
@tobrun tobrun deleted the android-embedding-fixes branch January 13, 2022 15:48
m0nac0 added a commit to maplibre/flutter-maplibre-gl that referenced this pull request May 13, 2022
flutter-mapbox-gl/maps#852

plus a temporary fix for the platform dependency (compile time error)

Co-Authored-By: Felix Horvat <24698503+felix-ht@users.noreply.github.com>
m0nac0 added a commit to maplibre/flutter-maplibre-gl that referenced this pull request May 13, 2022
* 710 (Get rid of MapboxGlPlatform.getInstance) flutter-mapbox-gl/maps#710
* 852( Android embedding fixes - migrate to maven) flutter-mapbox-gl/maps#852 was also required to fix compile time errors
* a small fix for the platform dependency
* Update CI flutter version
* cherry-pick upstream#904 (fix android build issues)

Co-authored-by: shroff <502644+shroff@users.noreply.github.com>
Co-authored-by: Felix Horvat <24698503+felix-ht@users.noreply.github.com>
mario-jerkovic pushed a commit to mario-jerkovic/flutter_mapbox_gl that referenced this pull request Sep 24, 2022
* Migrate maven repository from jcenter to mavenCentral

* Migrate maven repository from jcenter to mavenCentral

* migrate to maven
fix android embedding issue

Co-authored-by: luis felipe de oliveira <loliveira@inspectos.com>
(cherry picked from commit bd2ade0)
mario-jerkovic pushed a commit to mario-jerkovic/flutter_mapbox_gl that referenced this pull request Sep 24, 2022
* Migrate maven repository from jcenter to mavenCentral

* Migrate maven repository from jcenter to mavenCentral

* migrate to maven
fix android embedding issue

Co-authored-by: luis felipe de oliveira <loliveira@inspectos.com>
(cherry picked from commit bd2ade0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants