Author: brianf
Date: Wed Nov 14 05:26:34 2007
New Revision: 594871

URL: http://svn.apache.org/viewvc?rev=594871&view=rev
Log:
add banning of plugin snapshots

Modified:
    
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java

Modified: 
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java
URL: 
http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java?rev=594871&r1=594870&r2=594871&view=diff
==============================================================================
--- 
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java
 (original)
+++ 
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java
 Wed Nov 14 05:26:34 2007
@@ -208,7 +208,24 @@
             if ( !failures.isEmpty() )
             {
                 StringBuffer newMsg = new StringBuffer();
-                newMsg.append( "Some plugins are missing valid versions:\n" );
+                newMsg.append( "Some plugins are missing valid versions:" );
+                if (banLatest || banRelease || banSnapshots)
+                {
+                    newMsg.append ("(");
+                    if (banLatest)
+                    {
+                        newMsg.append ("LATEST ");   
+                    }
+                    if (banRelease)
+                    {
+                        newMsg.append ("RELEASE ");   
+                    }
+                    if (banSnapshots)
+                    {
+                        newMsg.append ("SNAPSHOT ");   
+                    }
+                    newMsg.append( "are not allowed )\n");
+                }
                 iter = failures.iterator();
                 while ( iter.hasNext() )
                 {
@@ -479,7 +496,7 @@
 
     protected boolean isSnapshot ( String baseVersion )
     {
-        return Artifact.VERSION_FILE_PATTERN.matcher( baseVersion ).matches();
+        return Artifact.VERSION_FILE_PATTERN.matcher( baseVersion ).matches() 
|| baseVersion.endsWith( Artifact.SNAPSHOT_VERSION );
     }
 
     /*


Reply via email to