-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Dagger 2 swallows errors #306
Comments
Can you post a small sample app that exhibits this? I'm not sure when the data binding "compilation" occurs, but my guess is that when javac fails due to Dagger, the entire gradle build halts and therefore you get these errors. Similar errors occur when you have just Dagger errors and the |
Sadly, I can't reproduce it in a separate project. :( Do you want a copy of the output of |
After couple of days of debugging, i found that gradle.projectsEvaluated {
tasks.withType(JavaCompile) {
options.compilerArgs << "-Xmaxerrs" << "500"
}
} so @ronshapiro you can close this issue. |
@ronshapiro @charbgr |
@charbgr You saved my life, can we document this somewhere? |
What errors were being displayed such that you had over 100 On Tue, 26 Jul 2016 at 22:38 Jignesh Sanghani notifications@github.com
|
@cgruber |
If you're using data binding, android studio generates error for every layout file even though there's no error in them, which causes dagger errors to be swallowed. |
This seems to be a periodic question, I'm going to add some docs to it so that it's more clear. |
The mentioned fix actually didn't work for me with Gradle 4.0. It does not find the allprojects {
afterEvaluate {
tasks.withType(JavaCompile.class) {
options.compilerArgs << "-Xmaxerrs" << "500"
}
}
} |
But for koltin it's showing databinding error in case of dagger related errors. Dagger related error is not showing in logs. Can anybody help me with this error |
@sangeetsuresh1 when annotation processor errors, the error output is mixed with errors coming from all annotation processors (including databinding). Increase the number of errors as mentioned above and search for possible dagger problems in that big list of errors. There is not really a better way at the moment. |
@tasomaniac I have found that new data binding library is causing the issue |
Can you guys tell us an update on this? Any time something is wrong I just give up and go with whats working. But it's hard to modularize when Dagger does not tell me whats wrong. |
@breakline87 it's not dagger swallowing the errors. It's the java compiler that by default cuts out after 100 errors. 100 errors is a lot normally but when you have a bunch of annotation processors like dagger and android databindings you easily pass that limit. It's not something that can (or probably even should) be fixed in dagger, the only thing you can do is increase the maximum number of displayed compilation errors as shown in multiple comments above (eg #306 (comment) or #306 (comment) ) |
Unfortunately, there is not always an appropriate error message. Several bugs in databinding can cause this (one example would be to use wrong package names in layout files). Dagger fails when I try to inject into a private field, however either I dont see the error message (at least not anymore) or its just not there. Sometimes it takes up hours to find a bug because all I see is "class not found" and similar error messages everywhere. Even if I see the whole output in a big project that means thousands of lines I dont care about at all. |
An advice that I can give you is to go one step at a time when it comes to dagger. Don't change bunch of things and try to run some time later. If you build each time after you do a dagger related change, then you will know what exactly broke it when it is broken. I know building also takes time but it will not take hours. |
Why it is not showing errors for new android data binding compiler i.e v2. Have somebody found the root cause of errors not showing? |
Kotlin docs helped me:
Of course this should be used if you use kapt |
@vfarafonov This isn't fixing the problem for us. Even after putting that configuration in our offending module and the app module, the build still fails with kapt 100 errors.
|
FWIW, @netdpb is working on limiting the amount of actual errors that Dagger prints out, and this should help greatly. Specifically, for many types of errors, we used to print the error N times, where N was the number of entry points from a (sub)component where the error was presented. Initially that was due to an implementation detail, but now it's not necessary and it proves to not be so helpful. So we're likely going to pick 1 of the entry points, print information for that. Hopefully that will make these types of problems less likely to come up. |
@benwicks With javac that would mean that you're using Java 10 (or 9) and didn't pass |
I've applied it to all my submodules with this in my root build.gradle:
|
@carlonzo Only solution that actually works for me. Thanks for sharing! |
Thank you |
thanks |
If you see many many errors coming from Android Databinding, upgrade to Android Gradle Plugin version 3.3.+ and it will show only 1 error. So this way you don't have to increase |
Hi, i have an unexpected problem with my setup. Also to inform you, i heavily use the official android databinding in my project (in 10+ activities and 20+ fragments).
my app build.gradle is something like this:
the root build.gradle is this one:
In my code, i make an enforced error(i know that i cannot create an object with the same method name in a module). The enforced error looks like this:
For some reason, when i build my project i get 100+ errors from databinding with this output:
cannot find symbol class BR
and dagger doesn't prints this error
Cannot have more than one @Provides method with the same name in a single module
Seems that Dagger "swallows" this error or the databinding overrides some behaviour of the annotation processor tool(apt).
Do you know what's going on?
The text was updated successfully, but these errors were encountered: