Skip to content

Use deprecated dependency until legacy renderer is removed#11185

Open
mboetger wants to merge 6 commits intoflutter:mainfrom
mboetger:remove-deprecated-library
Open

Use deprecated dependency until legacy renderer is removed#11185
mboetger wants to merge 6 commits intoflutter:mainfrom
mboetger:remove-deprecated-library

Conversation

@mboetger
Copy link
Contributor

@mboetger mboetger commented Mar 5, 2026

There is a crash due to play services removing the dependency for the legacy map renderer. This adds the necessary work around for users running into the crash.

Fixes: flutter/flutter#183087

Pre-Review Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] page, which explains my responsibilities.
  • I read and followed the [relevant style guides] and ran [the auto-formatter].
  • I signed the [CLA].
  • The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
  • I [linked to at least one issue that this PR fixes] in the description above.
  • I followed [the version and CHANGELOG instructions], using [semantic versioning] and the [repository CHANGELOG style], or I have commented below to indicate which documented exception this PR falls under[^1].
  • I updated/added any relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or I have commented below to indicate which [test exemption] this PR falls under[^1].
  • All existing and new tests are passing.

@mboetger mboetger marked this pull request as ready for review March 18, 2026 18:28
@mboetger mboetger requested a review from reidbaker as a code owner March 18, 2026 18:28
@mboetger mboetger requested a review from stuartmorgan-g March 18, 2026 18:28
@mboetger mboetger added the CICD Run CI/CD label Mar 18, 2026
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds the org.apache.http.legacy library to the AndroidManifest.xml to support the legacy Google Maps renderer on newer Android versions. This change is intended to fix a crash. The package version and changelog are also updated accordingly. A suggestion has been made to improve the changelog description for clarity.

Copy link
Collaborator

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

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

LGTM

mboetger and others added 3 commits March 18, 2026 13:57
…ELOG.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@mboetger mboetger added CICD Run CI/CD and removed CICD Run CI/CD labels Mar 18, 2026
@reidbaker reidbaker added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App CICD Run CI/CD p: google_maps_flutter platform-android

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Google Maps Flutter plugin crashes on Android due to missing org.apache.http.legacy library

3 participants