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

[firebase_messaging] readme update per issue resolution flutter/flutter/issues/38459 #12

Closed
wants to merge 6 commits into from

Conversation

mictay
Copy link

@mictay mictay commented Aug 23, 2019

Description

Using the latest firebase_messaging breaks gradle builds. Updating the documentation in the readme.md file aligns with the https://firebase.google.com/docs/cloud-messaging/android/client setup instructions that resolves the flutter gradle build issue.

Related Issues

flutter/flutter#38459

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
  • All existing and new tests are passing.
  • I updated/added relevant documentation (doc comments with ///).
  • The analyzer (flutter analyze) does not report any problems on my PR.
  • I read and followed the Flutter Style Guide.
  • The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
  • [ x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
  • I updated CHANGELOG.md to add a description of the change.
  • I signed the CLA.
  • I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

  • Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
  • [ x] No, this is not a breaking change.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@collinjackson
Copy link
Contributor

Thanks for the PR! Please sign the CLA, and update the pubspec.yaml and CHANGELOG.

@collinjackson collinjackson changed the title readme update per issue resolution flutter/flutter/issues/38459 [firebase_messaging] readme update per issue resolution flutter/flutter/issues/38459 Aug 23, 2019
@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

Copy link
Contributor

@collinjackson collinjackson left a comment

Choose a reason for hiding this comment

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

I took a closer look and I am unsure that this is the right approach.

Other Flutterfire plugins are specifying mostly just API dependencies; the implementation dependency is just 'com.google.firebase:firebase-common:16.1.0' and that seems to be enough for gradle to resolve the rest of the Firebase SDK.

The continuous integration is green and that means that the building the all_plugins test app is working. Something is different about what you're doing and what we're doing. I think the culprit is probably implementation 'com.google.firebase:firebase-core:17.0.0' in your app-level build.gradle... where did that come from? When I try combining that line with a dependency on firebase_messaging plugin I get similar build errors:

FAILURE: Build failed with an exception.                                
                                                                        
* What went wrong:                                                      
Could not determine the dependencies of task ':app:compileReleaseJavaWithJavac'.
> In project 'app' a resolved Google Play services library dependency depends on another at an exact version (e.g. "[18.0.
  0]", but isn't being resolved to that version. Behavior exhibited by the library will be unknown.
                                                                        
  Dependency failing: com.google.firebase:firebase-messaging:18.0.0 -> com.google.firebase:firebase-iid@[18.0.0], but fire
  base-iid version was 19.0.0.                                          
                                                                        
  The following dependencies are project dependencies that are direct or have transitive dependencies that lead to the art
  ifact with the issue.                                                 
  -- Project 'app' depends onto com.google.firebase:[email protected]
  -- Project 'app' depends on project 'firebase_messaging' which depends onto com.google.firebase:[email protected].
  0                                                                     
  -- Project 'app' depends onto com.google.firebase:[email protected]
  -- Project 'app' depends on project 'firebase_core' which depends onto com.google.firebase:[email protected]
  -- Project 'app' depends onto com.google.firebase:[email protected]
  -- Project 'app' depends onto com.google.android.gms:[email protected]
  -- Project 'app' depends onto com.google.firebase:[email protected] 
                                                                        
  For extended debugging info execute Gradle from the command line with ./gradlew --info :app:assembleDebug to see the dep
  endency paths to the artifact. This error message came from the google-services Gradle plugin, report issues at https://
  github.com/google/play-services-plugins and disable by adding "googleServices { disableVersionCheck = false }" to your b
  uild.gradle file.                                                     
                                                                        
* Try:                                                                  
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
                                                                        
* Get more help at https://help.gradle.org                              
                                                                        
BUILD FAILED in 3s  

Perhaps we should be discouraging developers from specifying any implementation versions of Firebase libraries rather than adding more specific implementation dependencies as you're doing in this PR?

/cc @mklim @kroikie in case they have further thoughts

@kroikie
Copy link
Collaborator

kroikie commented Nov 15, 2019

@mictay Thank for the PR but the version of the native SDK should be determined by the plugin, if there is an issue the plugin should be updated to use the correct version of the SDK.

@firebase firebase locked and limited conversation to collaborators Aug 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
blocked: customer-response Waiting for customer response, e.g. more information was requested. cla: yes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants