Skip to content

Roll pub packages - #148245

Closed
flutter-pub-roller-bot wants to merge 1 commit into
flutter:masterfrom
flutter-pub-roller-bot:packages-autoroller-branch-83
Closed

flutter-pub-roller-bot wants to merge 1 commit into
flutter:masterfrom
flutter-pub-roller-bot:packages-autoroller-branch-83

Conversation

@flutter-pub-roller-bot

Copy link
Copy Markdown
Contributor

This PR was generated by flutter update-packages --force-upgrade.

@flutter-pub-roller-bot flutter-pub-roller-bot added tool Affects the "flutter" command-line tool. See also t: labels. autosubmit Merge PR when tree becomes green via auto submit App labels May 13, 2024
@github-actions github-actions Bot removed the tool Affects the "flutter" command-line tool. See also t: labels. label May 13, 2024
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label May 13, 2024
@auto-submit

auto-submit Bot commented May 13, 2024

Copy link
Copy Markdown
Contributor

auto label is removed for flutter/flutter/148245, due to - The status or check suite Mac tool_integration_tests_4_4 has failed. Please fix the issues identified (or deflake) before re-applying this label.

@goderbauer

Copy link
Copy Markdown
Member

Looks like the roller encounter a problem with rolling the latest version of the camera plugin into the repository:

Plugin camera:android has conflicting direct dependency implementations:
            [        ]              camera_android
            [        ]              camera_android_camerax
            [        ]            To fix this issue, remove all but one of these dependencies from pubspec.yaml.

See https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8748029226655661665/+/u/run_test.dart_for_tool_integration_tests_shard_and_subshard_3_4/stdout#:~:text=46%20ms%5D%20Plugin-,camera%3Aandroid,-has%20conflicting%20direct

@bparrishMines Could you take a look at this?

@bparrishMines

Copy link
Copy Markdown
Contributor

This should be caused by flutter/packages#6629

cc @camsim99 @stuartmorgan

@bparrishMines

Copy link
Copy Markdown
Contributor

We should just need to remove camera_android from the list. The default Android implementation is now camera_android_camerax.

@stuartmorgan-g

Copy link
Copy Markdown
Contributor

Yes, it's invalid for an application to depend directly on two different implementations, so this pubspec needs to list only one of the two dependencies.

@goderbauer

Copy link
Copy Markdown
Member

Thank you. Created a manual roll in #148426. I am going to close this PR once that other PR has been submitted to get the auto-roller back to rollin'.

auto-submit Bot pushed a commit that referenced this pull request May 20, 2024
@goderbauer

Copy link
Copy Markdown
Member

Closing this to restart the roller as #148426 has landed.

@goderbauer goderbauer closed this May 21, 2024
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.

5 participants