-
Notifications
You must be signed in to change notification settings - Fork 612
A problem was found with the configuration of task ':app:uploadCrashlyticsMappingFileRelease' (type 'UploadMappingFileTask'). - Type 'UploadMappingFileTask' property 'googleServicesResourceRoot' doesn't have a configured value. #2721
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
Comments
I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight. |
Any support? |
Help please |
We cannot compile/build a release aab with this update version due to this error. |
Hello, I'm affected as well since upgrading the Firebase Crashlytics Gradle plugin from 2.6.1 to 2.7.0. Looks like that 2.7.0 version didn't undergo end-to-end testing… I have no choice but to revert the upgrade. |
Same issue here as well after upgrade to 2.7.0 |
same issue, using gradle 7.0.2 |
The tests probably did not find this issue since the firebase repo is not on Gradle 7 yet. It's actually on Gradle 5.x |
Gradle 5 is like 2 years old now? Maybe it's time to move to Gradle 6. Testing on latest Gradle release would be nice too. |
Hi @ArcherEmiya05 thanks reporting, we're currently looking into this. Quick question, may we know what Gradle version you are using when the error occurs? Also I tried the quickstart app with 7.0.1 Gradle Version and crashlytics-gradle:2.7.0 but I was not able to reproduce the issue. It would be great if you can provide a sample project where the issue is reproducible. Thanks |
Using Gradle 7.0.2 in my case. |
Gradle Plugin Gradle version |
This public PR: CruGlobal/godtools-android#1733 is having the same issue with the update (the currently failing GitHub Action check) |
It happens on gradle 6.7.1 too |
Same to me |
It happens on Gradle 6.9 also, with android gradle plugin 4.2.1 |
My bad, I was actually on 2.7.0. So yes, that seems to be the culprit. |
Per checking it seems to be a compatibility issue with the Android Gradle plugin, though not 100% guaranteed since I can't reproduce locally. Let me assign for now. @mrichards could you have a look at this? (looks like a similar issue reported via support portal b/186748865). |
Found a fix for me. This produces the error
While this does not, note com.google.gms.google-services is now ABOVE com.google.firebase.crashlytics
When you update to com.google.firebase:firebase-crashlytics-gradle:2.7.0 and sync the changes with them in the wrong order you are given an message stating that is the fix.
|
This does not sound like a fix, it's just causing a crash a bit earlier, before crashlytics plugin configuration will try to get missing property. |
Sorry my comment might not have been clear, if you put com.google.firebase.crashlytics before com.google.gms.google-services you get that warning. (then if you try to build your project you get the error) If you put it how it tells you, then you don't get the warning or the build error. i have edited my reply to make it clear, in the wrong order you get the warning |
I can verify that reordering the order that the google plugins are applied solved the issue for me. |
There's a |
Yes, @dmessano has it correct. As of 2.7.0, the order in which the plugins are applied is now relevant; this (ironically) is a side-effect of a change that will make it easier for customers to avoid using the GMS plugin if they choose. It also fixed a configuration cache issue in Gradle 7. Are some builds not displaying the logged error with the directions on how to fix? ("Make sure com.google.gms.google-services is applied BEFORE com.google.firebase.crashlytics.") We were hoping this message would be sufficient to prevent support requests, but if it is not printing for folks, please let me know as I'd like to understand how to ensure it gets produced for your specific configuration. |
It's not showing up for me at least. Did you test the error cases on your end to ensure the proper message is shown instead of another puzzling failure occuring before? |
I've changed the build file and put this in place: plugins { Now LINT complains with the following warning: "Please apply google-services plugin at the bottom of the build file." What to do? |
@peterhav Try putting the crashlytics plugin at the very bottom of the plugin list and the google-services plugin right above that. |
Thanks, that worked. I now have:
|
The message is logged as a warning, not an error, because in some cases the build should be able to continue successfully. We'll consider changing the logic a bit to ensure it is logged as an error if the end result will be a failed build. I'm going to close this issue as the root cause has been identified and the original issue is resolved. Please re-open if you believe this is in error. |
In which version has the issue been resolved? |
After updating to
classpath 'com.google.firebase:firebase-crashlytics-gradle:2.7.0'
I tried to read the documentation and changelog, however none of them help on how to fix the problem.
The text was updated successfully, but these errors were encountered: