You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Stephen Connolly (JIRA)" <ji...@apache.org> on 2017/06/22 19:54:01 UTC
[jira] [Updated] (MENFORCER-228) DependencyConvergence: Simplify
logging errors
[ https://issues.apache.org/jira/browse/MENFORCER-228?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Stephen Connolly updated MENFORCER-228:
---------------------------------------
Fix Version/s: 3.0.0
> DependencyConvergence: Simplify logging errors
> ----------------------------------------------
>
> Key: MENFORCER-228
> URL: https://issues.apache.org/jira/browse/MENFORCER-228
> Project: Maven Enforcer Plugin
> Issue Type: Improvement
> Components: Standard Rules
> Affects Versions: 1.4
> Environment: Maven 3.2.2; Oracle JDK 8
> Reporter: Stephan Krull
> Assignee: Karl Heinz Marbaise
> Priority: Minor
> Fix For: 1.4.2, 3.0.0
>
>
> The dependency convergence rule check works great. I just wonder if it is really necessary to log the according error messages two times:
> see source code http://svn.apache.org/viewvc/maven/enforcer/tags/1.4/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/DependencyConvergence.java?view=markup
> {code:title=lines 127 to 134}
> for ( CharSequence errorMsg : errorMsgs )
> {
> log.warn( errorMsg );
> }
> if ( errorMsgs.size() > 0 )
> {
> throw new EnforcerRuleException( "Failed while enforcing releasability the error(s) are " + errorMsgs );
> }
> {code}
> this produces outputs like:
> {noformat}
> [INFO] --- maven-enforcer-plugin:1.4:enforce (enforce dependency convergence) @ bom ---
> [WARNING]
> Dependency convergence error for commons-logging:commons-logging:1.1.3 paths to dependency are:
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-org.springframework:spring-context:4.0.9.RELEASE
> +-org.springframework:spring-core:4.0.9.RELEASE
> +-commons-logging:commons-logging:1.1.3
> and
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-com.orientechnologies:orientdb-graphdb:2.1-rc1
> +-com.tinkerpop.blueprints:blueprints-core:2.6.0
> +-commons-logging:commons-logging:1.1.1
> [WARNING]
> Dependency convergence error for org.slf4j:jcl-over-slf4j:1.7.6 paths to dependency are:
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-org.activiti:activiti-engine:5.17.0
> +-org.activiti:activiti-bpmn-converter:5.17.0
> +-org.slf4j:jcl-over-slf4j:1.7.6
> and
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-org.activiti:activiti-engine:5.17.0
> +-org.slf4j:jcl-over-slf4j:1.7.6
> and
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-org.springframework.ldap:spring-ldap-core:2.0.2.RELEASE
> +-org.springframework.data:spring-data-commons:1.6.1.RELEASE
> +-org.slf4j:jcl-over-slf4j:1.7.1
> [WARNING] Rule 0: org.apache.maven.plugins.enforcer.DependencyConvergence failed with message:
> Failed while enforcing releasability the error(s) are [
> Dependency convergence error for commons-logging:commons-logging:1.1.3 paths to dependency are:
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-org.springframework:spring-context:4.0.9.RELEASE
> +-org.springframework:spring-core:4.0.9.RELEASE
> +-commons-logging:commons-logging:1.1.3
> and
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-com.orientechnologies:orientdb-graphdb:2.1-rc1
> +-com.tinkerpop.blueprints:blueprints-core:2.6.0
> +-commons-logging:commons-logging:1.1.1
> ,
> Dependency convergence error for org.slf4j:jcl-over-slf4j:1.7.6 paths to dependency are:
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-org.activiti:activiti-engine:5.17.0
> +-org.activiti:activiti-bpmn-converter:5.17.0
> +-org.slf4j:jcl-over-slf4j:1.7.6
> and
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-org.activiti:activiti-engine:5.17.0
> +-org.slf4j:jcl-over-slf4j:1.7.6
> and
> +-my.group:my.artifact:2.1-SNAPSHOT
> +-org.springframework.ldap:spring-ldap-core:2.0.2.RELEASE
> +-org.springframework.data:spring-data-commons:1.6.1.RELEASE
> +-org.slf4j:jcl-over-slf4j:1.7.1
> ]
> {noformat}
> the warning log messages are identical to the messages of the exception. so just skip one of the outputs if possible.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)