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
Fix the incompatibility issue between Analytics and Messaging #319
Conversation
Error from CI:
Basically, somewhere there is a dependency on a support library 28.0.0.1 and 28.0.0.3. I think Xamarin.GooglePlayServices.Basement is referencing an old versions somehow... @moljac might know offhand why the versions are not matching in this build... |
During repo consolidation attempts I have set (or tried to set) all dependencies in https://github.com/xamarin/GooglePlayServicesComponents/blob/master/config.json#L693-L732 Smells like some transitive dependency pulls 28.0.0.1 version[s]. Not sure. Wild guess. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
due to the fact that dotnet
does not have reinstall concept. I use this trick for installation/reinstallation:
dotnet tool uninstall --global Cake.Tool
dotnet tool install --global Cake.Tool
This way users are safe with the newest versions of the tools used to build the repo.
Othrewise approved.
Pushed a new commit to fix Xamarin.GooglePlayServices.Ads |
@@ -613,13 +637,13 @@ | |||
"version" : "17.0.5", | |||
"nugetVersion" : "71.1705.0", | |||
"nugetId" : "Xamarin.Firebase.Measurement.Connector.Impl", | |||
"dependencyOnly" : false | |||
"dependencyOnly" : true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is Firebase package, so we must bind it. It should be
"dependencyOnly" : false
@@ -461,7 +485,7 @@ | |||
"version" : "16.3.0", | |||
"nugetVersion" : "71.1630.0", | |||
"nugetId" : "Xamarin.Firebase.Analytics.Impl", | |||
"dependencyOnly" : false | |||
"dependencyOnly" : true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is Firebase package, so we must bind it. It should be
"dependencyOnly" : false
Meaning of
"dependencyOnly" : true
is that it is not bound in. this repo, but there is dependency nuget. Dependency nuget could have jar
/aar
as payload (reference jar) which is not bound, but we don't do that. You will see all of "dependencyOnly" : true
are external packages (Android.Support and other mostly from XamarinComponents repo, TensorFlow, etc...)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand it is Impl
, so maybe not relevant for bindings, but our internal convention was to bind those too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes the only issue what that the binderator was complaining for a superior version But no new version have been Feb, 2019.
What should be the good approach for that kind of situation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
binderator was complaining for a superior version
This usually indicates something. OK. I have forked and am building. Will raise you PR (I hope)
This is crazy:
so:
and nuspec looks like this:
Where does 28.0.0.1 come from? |
@GuillaumeSE
I will check it and fix it with other stuff. |
Thanks! I'm investigating on my side with part I did wrong!. |
Sorry for delay, but I have to make master green with new tooling. As soon as I finish that I'll check this PR. I hope this is OK with you. |
@moljac Is there any update for this? Can you give a rough estimation when you expect to have this fixed? |
These versions predate AndroidX, so I'm guessing this is no longer relevant. |
Google Play Services Version (eg: 8.4.0):
V9.0
Does this change any of the generated binding API's?
Yes
Describe your contribution
Fix the incompatibility Issue between Analytics and Messaging by upgrading the proper dependencies.
FirebaseAnalytics & FirebaseMessaging, version mismatch on gms.measurement.base #280
"Seal" the sub-dependencies with seams to do not evolve anymore according to maven last update. This will help the Android-Binderator to complete his Dependencies check