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 ); } /*