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

display-dependency-updates doesn't manage transitive dependencies #267

Closed
ghusta opened this issue Apr 6, 2018 · 4 comments
Closed

display-dependency-updates doesn't manage transitive dependencies #267

ghusta opened this issue Apr 6, 2018 · 4 comments

Comments

@ghusta
Copy link
Contributor

ghusta commented Apr 6, 2018

Only the direct dependencies are managed in this goal. The transitive ones are not.

For example :

When running mvn versions:display-dependency-updates on a project with the direct dependency org.apache.struts:struts-core:1.3.10, commons-beanutils:commons-beanutils (transitive) is not reported whereas there is a newer version than 1.8.0 (latest is 1.9.3).

Enhancement :

Maybe a parameter like processTransitiveDependencies could be useful (default false for backward compatibility).

@ghusta
Copy link
Contributor Author

ghusta commented Apr 6, 2018

A good starting point could be to use org.apache.maven.shared.artifact.filter.collection.ProjectTransitivityFilter in org.apache.maven.shared:maven-common-artifact-filters:3.0.1.

Like in : https://github.com/apache/maven-dependency-plugin/blob/maven-dependency-plugin-3.1.0/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractDependencyFilterMojo.java#L285

@hemanthacharya
Copy link

Yes, this would very useful.

@ghusta
Copy link
Contributor Author

ghusta commented Jun 28, 2019

The parameter processDependencyManagementTransitive belongs to the goal dependency-updates-report.
It's not present in display-dependency-updates.

There's also a mention of this parameter here : #291, still unresolved

@apoorva-sriv
Copy link

apoorva-sriv commented Aug 30, 2022

This issue should be closed since transitive dependencies are being reported now, and parameter processDependencyManagementTransitive has been added since 2.11 and #588.

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

4 participants