You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by br...@apache.org on 2005/07/28 17:45:14 UTC
svn commit: r225803 -
/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/WarningResolutionListener.java
Author: brett
Date: Thu Jul 28 08:45:11 2005
New Revision: 225803
URL: http://svn.apache.org/viewcvs?rev=225803&view=rev
Log:
PR: MNG-670
tone it down a bit
Modified:
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/WarningResolutionListener.java
Modified: maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/WarningResolutionListener.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/WarningResolutionListener.java?rev=225803&r1=225802&r2=225803&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/WarningResolutionListener.java (original)
+++ maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/WarningResolutionListener.java Thu Jul 28 08:45:11 2005
@@ -19,6 +19,9 @@
import org.apache.maven.artifact.Artifact;
import org.codehaus.plexus.logging.Logger;
+import java.util.HashSet;
+import java.util.Set;
+
/**
* Send resolution warning events to the warning log.
*
@@ -30,6 +33,8 @@
{
private Logger logger;
+ private static Set ignoredArtifacts = new HashSet();
+
public WarningResolutionListener( Logger logger )
{
this.logger = logger;
@@ -61,10 +66,15 @@
public void updateScopeCurrentPom( Artifact artifact, String scope )
{
- logger.warn( "\n\tArtifact " + artifact.getId() + " is having scope '" + artifact + "' replaced with '" +
- scope + "'\n" +
- "\tas a dependency has given a broader scope. If this is not intended, use -X to locate the dependency,\n" +
- "\tor force the desired scope using dependencyManagement.\n" );
+ // TODO: better way than static? this might hide messages in a reactor
+ if ( !ignoredArtifacts.contains( artifact ) )
+ {
+ logger.warn( "\n\tArtifact " + artifact.getId() + " has scope '" + artifact.getScope() +
+ "' replaced with '" + scope + "'\n" +
+ "\tas a dependency has given a broader scope. If this is not intended, use -X to locate the dependency,\n" +
+ "\tor force the desired scope using dependencyManagement.\n" );
+ ignoredArtifacts.add( artifact );
+ }
}
public void updateScope( Artifact artifact, String scope )
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org