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

Roll pub packages #148245

Conversation

flutter-pub-roller-bot
Copy link
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
Copy link
Contributor

auto-submit bot commented May 13, 2024

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
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
Contributor

This should be caused by flutter/packages#6629

cc @camsim99 @stuartmorgan

@bparrishMines
Copy link
Contributor

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

@stuartmorgan
Copy link
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
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
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.

None yet

5 participants