Only print relocation warnings in standard output for the current pom

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0
  • Fix Version/s: 2.0.2
  • Component/s: Logging
  • Labels:
    None
  • Complexity:
    Intermediate

Description

Relocation warnings of transitive dependencies shouldn't be printed in standard output because people gets confused, ask questions and try to change other people poms although they work correctly

We should only print the warnings of the current pom so the author can improve it.

  1. Text File
    MNG-1689-maven-artifact.patch
    19/Dec/05 2:06 AM
    0.8 kB
    Edwin Punzalan
  2. Text File
    MNG-1689-maven-project.patch
    19/Dec/05 2:06 AM
    1 kB
    Edwin Punzalan

Issue Links

supercedes

Bug - A problem which impairs or prevents the functions of the product. MEV-228 dom4j needs xml-apis updated

  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Bug - A problem which impairs or prevents the functions of the product. MEV-265 Commons Digester has invalid version number specified for xml-apis

  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Edwin Punzalan added a comment -

Attached patch will print the project transitive relocated dependencies as debug, and print direct dependencies as warn.

Show
Edwin Punzalan added a comment - Attached patch will print the project transitive relocated dependencies as debug, and print direct dependencies as warn.
Hide
Permalink
Carlos Sanchez added a comment -

The isProjectDependency method in patch always returns false because project.getDependencies() is empty

Show
Carlos Sanchez added a comment - The isProjectDependency method in patch always returns false because project.getDependencies() is empty
Hide
Permalink
Edwin Punzalan added a comment -

Sorry about the previous patch...

The attached patches will fix the issue. Tested it on my machine to work.

Show
Edwin Punzalan added a comment - Sorry about the previous patch... The attached patches will fix the issue. Tested it on my machine to work.

People

  • Assignee:
    Carlos Sanchez
    Reporter:
    Carlos Sanchez
Vote (1)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: