-
-
Notifications
You must be signed in to change notification settings - Fork 114
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
Could not resolve all task dependencies for configuration #1108
Comments
Just for the record, I've now submitted a bug report for the solution stated by the official documentation. Hopefully it will get fixed, but it might still be worth figuring out why the other setup mentioned above doesn't work. |
I think that other configuration you create needs to have some metadata attached to it. If you provide a reproducer I can take a look and see if there's something I can do in DAGP. |
@autonomousapps here we go: Thanks! |
I've just resolved another issue that looks at least superficially similar. Could you try the latest snapshot (wait at least an hour from when I post this message)? |
Hey @autonomousapps, Thanks for the update. Unfortunately, I've tested |
Alright, thanks! Looks like something that might be Android-specific. |
I was able to workaround this by adding this to the top of the "wrapper" module: // wrapper-module/build.gradle
plugins {
id("java")
} I find the existence of such modules unusual -- what's the use-case? Why not just add the jar as a dependency to "normal" modules directly? |
For Android projects, it throws a lint error since adding AAR files directly misses the non-JVM parts of the archive. |
What exactly throws a lint error? |
Sorry doesn't look like I can get the same error as before. Now I get the following trying to sync the project after adding 1.32 version of the plugin:
|
@bidrohi this looks like a different issue. Could you file a new report? |
We have an AAR-file that we have to import to our build somehow in order to be able to use it. We're okay about changing the way we import it (as long as it's reliable), but I'm not sure how you would add that as a dependency directly? Most other methods I've found is either outdated or not stable to use, unfortunately. |
Does a workaround like this help? Alternatively, you could publish that AAR to an internal artifact repo and consume it that way. |
Yeah, sorry forgot to give an answer here. I tried that, but I then wasn't able to access the content of the aar-file (imports of classes wasn't found anymore). It might be possible to do something like that, but I'm not an expert in gradle unfortunately.
Might be an idea we could test out a bit. We are however several developers working on this project so we would need to have some automatic setup for this 🤔 |
This works for me:
|
This looks much more "normal" to me! |
Sorry.. I didn't share the lint error before:
|
Looks like it was broken with Kotlin 1.9.0 updating to 1.9.24 resolved this issue. |
The issue with this solution is that it unfortunately gives and error when linting as mentioned in the original post. I have submitted an issue (see first answer), but I unfortunately don't think that will be fixed any time soon. |
Plugin version
1.28.0 (and 1.29.0)
Gradle version
8.2
JDK version
17
(Optional) Kotlin and Kotlin Gradle Plugin (KGP) version
Kotlin 1.9.21
(Optional) Android Gradle Plugin (AGP) version
8.2.1
Describe the bug
After bumping to 1.28.0 we get this error when building the project (for instance via
build
):moduleA
is here a regular module (with plugincom.android.library
) andmoduleB
is a "wrapper" around an AAR file to be able to use it. The AAR file is included like this:This is not how the official documentation states it, but due to a weird bug this doesn't work for other reasons unfortunately.
I don't know if we can fix this in any other ways (is it possible to ignore dependencies or modules somehow?), so wanted to file this bug if you weren't aware of this. And for the record, it works fine for version 1.27.0.
The text was updated successfully, but these errors were encountered: