-
Notifications
You must be signed in to change notification settings - Fork 10
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
Circular task dependency with createORMLiteConfigFileRelease #23
Comments
We don't have this issue. But we don't use kotlin. I am not sure we can
help without more details.
Can you reproduce the bug in a sample project ?
2018-01-12 13:16 GMT-08:00 Winson Chiu <[email protected]>:
… We were previously on 1.0.13 and upgraded to 3.0.3 along side an AGP 3.0
upgrade.
Not sure what's causing this though.
FAILURE: Build failed with an exception.
* What went wrong:
Circular dependency between the following tasks:
:android:compileReleaseJavaWithJavac
+--- :android:compileReleaseKotlin
| +--- :android:kaptReleaseKotlin
| | \--- :android:kaptGenerateStubsReleaseKotlin
| | \--- :android:processReleaseResources
| | \--- :android:mergeReleaseResources
| | \--- :android:fabricGenerateResourcesRelease
| | \--- :android:mergeReleaseAssets
| | \--- :android:createORMLiteConfigFileRelease
| | +--- :android:compileReleaseJavaWithJavac (*)
| | \--- :android:compileReleaseKotlin (*)
| \--- :android:processReleaseResources (*)
+--- :android:generateReleaseSources
| \--- :android:processReleaseResources (*)
+--- :android:javaPreCompileRelease
| \--- :android:compileReleaseKotlin (*)
+--- :android:kaptReleaseKotlin (*)
\--- :android:processReleaseResources (*)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#23>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABv33ctR8RlK_zf3FpaTbNbQm5KiLyYIks5tJ8uVgaJpZM4Rc3T4>
.
|
Yes, I can reproduce in a sample. It seems this is actually a conflict with the Fabric plugin. If you create a new project in AS with all the defaults and replace the top of app's build.gradle with this, it'll occur:
|
Also, semi-related, but I noticed the plugin has a explicit dependency on 3.0.0-beta7. That actually makes the default project not resolve given your snippet in the README. You actually need the jcenter() and google() repositories in the block. |
Got same issue. Have you found a solution? |
One simple work around could be to separate the code that uses fabric in a new module. That's what we currently do at Groupon. As the fabrics plugin always triggers a change in the resources on every build, by isolating it in a small module we make our builds more cacheable, especially on CI. You can have a module architecture like this:
|
I would gladly merge a PR that would update the GAP version. |
Is there another solution to this Issue? |
The 9th September 2018 Google Firebase is planing to stop maintaining its current Crash Reporting Library and is beginning to use Crashlytics for it. As Crashlytics uses Fabric, this issue will happen more an more frequently. I cant wait more to be solved and I'm planning to migrate to Kripton Orm which first doesn't use Fabric, and second doesn't use a gradle plugin but annotationProcessor. Besides Kripton strengths makes it a better choice given my app arquitecture. |
I confirm, I can't use 3.0.4 because of this circular dependency problem either. |
Hey, is there a plan to fix this? |
Hi guys,
This plugin is not maintained. Feel free to submit PRs and a test /sample
to show it works.
I don't have time to fix it myself.
S.
…On Sun., Apr. 21, 2019, 07:02 Alon Vertzberger, ***@***.***> wrote:
Hey, is there a plan to fix this?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#23 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAN7PXMFFBA6VKKE2LLK5TLPRRXYXANCNFSM4ELTOT4A>
.
|
We were previously on 1.0.13 and upgraded to 3.0.3 along side an AGP 3.0 upgrade.
Not sure what's causing this though.
The text was updated successfully, but these errors were encountered: