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.

Sign up for GitHub

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

Jump to bottom

display-dependency-updates with processDependencyManagementTransitive = false ignores dependencyManagement in enabled profiles #1191

Closed
facboy opened this issue Nov 27, 2024 · 2 comments · Fixed by #1193
Closed

display-dependency-updates with processDependencyManagementTransitive = false ignores dependencyManagement in enabled profiles #1191

facboy opened this issue Nov 27, 2024 · 2 comments · Fixed by #1193
Assignees
@andrzejj0
Labels
bug
Milestone
2.19.0

Comments

@facboy
Copy link

facboy commented Nov 27, 2024

When running display-dependency-updates with processDependencyManagementTransitive is disabled (i.e. false), dependencyManagement in enabled profiles is not checked for updates.
@andrzejj0
Copy link
Contributor

Does it work with the precious version of the Plugin? Can you share a log with -X enabled?

Minimal pom.xml to reproduce?

@andrzejj0
Copy link
Contributor

From what I see, this has never worked correctly since the introduction of processDependencyManagementTransitive.

Reproduced.

@andrzejj0 andrzejj0 added the bug label Nov 27, 2024
@andrzejj0 andrzejj0 self-assigned this Nov 27, 2024
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Nov 27, 2024
@andrzejj0
Fixed mojohaus#1191: Processes (all) profiles from original model if …
e95f729 
…transitive is false
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Nov 27, 2024
@andrzejj0
Fixed mojohaus#1191: Processes (all) profiles from original model if …
491517b 
…transitive is false
@andrzejj0 andrzejj0 linked a pull request Nov 27, 2024 that will close this issue
Fixed #1191: Processes (all) profiles from original model if transitive is false #1193
Merged
@slawekjaranowski slawekjaranowski added this to the 2.19.0 milestone Nov 28, 2024
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Nov 28, 2024
@andrzejj0
Fixed mojohaus#1191: Processes (all) profiles from original model if …
b3c8af8 
…transitive is false
andrzejj0 added a commit to andrzejj0/versions-maven-plugin that referenced this issue Nov 30, 2024
@andrzejj0
Revert "Fixed mojohaus#1191: Processes (all) profiles from original m…
432a02d 
…odel if transitive is false (mojohaus#1193)"

This reverts commit a6f4ffb.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andrzejj0 andrzejj0

Labels
bug
Projects
None yet
Milestone
2.19.0
3 participants
@facboy @andrzejj0 @slawekjaranowski