Skip to content
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

aggregate-add-third-party: Many warnings about unused dependencies #334

Closed
digulla opened this issue May 10, 2019 · 3 comments
Closed

aggregate-add-third-party: Many warnings about unused dependencies #334

digulla opened this issue May 10, 2019 · 3 comments
Milestone

Comments

@digulla
Copy link

digulla commented May 10, 2019

I have a multi-module project. One module depends on org.apache.httpcomponents:httpclient:4.1.2 which uses the generic string "Apache License" in the POM (it's APLv2).

In order to get the correct version of the license in our legal reports, I added an override URL to the plugin configuration.

I'm now getting a dozen repetitions of this:

[WARNING] dependency [org.apache.httpcomponents--httpmime--4.1.2] does not exist in project.
[WARNING] dependency [org.apache.httpcomponents--httpcore--4.1.2] does not exist in project.
[WARNING] dependency [org.apache.httpcomponents--httpclient-cache--4.1.2] does not exist in project.
[WARNING] dependency [org.apache.httpcomponents--httpclient--4.1.2] does not exist in project.

I don't think this warning makes sense for the goal aggregate-add-third-party.

When using overrideUrl with a classpath resource, I might not make sense for other reports as well. Scenario: There is a company-wide file with overrides that have been cleared with the legal department. Almost no company project will use all dependencies in their projects.

I think it's being printed by DefaultThirdPartyTool.overrideLicenses() and maybe in DefaultThirdPartyTool.loadUnsafeMapping().

Not sure how to fix; it could be suppressed when an aggregated report is being run or with a config flag.

@ppalaga
Copy link
Contributor

ppalaga commented May 10, 2019

Could you please look whether this one is not a duplicate of #201 and eventually close the current issue?

@ppalaga
Copy link
Contributor

ppalaga commented May 10, 2019

BTW there is a PR for #201 with unresolved review requests #300 You may want to check whether the fix works for you and eventually add the missing parts so that it can be merged.

@digulla
Copy link
Author

digulla commented May 10, 2019

Duplicate of #201
Note: #201 talks about another but related error message logged by DefaultThirdPartyTool.loadUnsafeMapping()

@digulla digulla closed this as completed May 10, 2019
@ppalaga ppalaga added this to the 2.0.0 milestone Jun 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants