-
Notifications
You must be signed in to change notification settings - Fork 18
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
Allow Dependencies from Provided and Other Configurations Into Resulting Artifact #31
Comments
Correct me if I'm wrong, this was on a web-app, where the provided configuration was created by the war plugin and not by the extra-configurations-plugins. Is this a core gradle bug? |
+1 |
This is the behavior in gradle. So we emulate exactly what the
providedCompile from the war plugin does. I wouldn't consider it a bug.
On Mon, Feb 6, 2017 at 4:12 PM Justin Ryan <[email protected]> wrote:
Correct me if I'm wrong, this was on a web-app, where the provided
configuration was created by the war plugin and not by the
extra-configurations-plugins. Is this a core gradle bug?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#31 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAvvFqvGHBCQgaR7wQeMl-rf-sqOkc29ks5rZ7bPgaJpZM4L42h1>
.
|
The behavior I've seen is as follows:
@sghill @quidryan can you confirm if this matches what you have seen? if so, at the very least the documentation for "provided" should explicitly call this out. It seems to me that the transitive dependencies of Y should be included in my war, regardless of whether X happens to have transitive dependency overlap. |
That is the behavior that our provided performs just like it is the behavior that providedCompile from the war plugin performs. You have told gradle that dependency X and all of its transitives will be provided, since why would X be provided by the system but not A, B, and C. I would be happy erroring or warning on the build that you have specified something that can't be resolved. |
Confirming this is how @Helmsdown Yes, that matches what I have seen. I also like @rspieldenner's idea of erroring or warning on the build since this behavior is almost never what is intended. There is another scenario that may need a new configuration:
|
+1 |
If a dependency is declared as
provided
and another configuration that is included in resulting artifacts (such ascompile
orruntime
), it should not remove the dependency from the artifact.The text was updated successfully, but these errors were encountered: