You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2011/03/19 22:27:45 UTC
svn commit: r1083296 -
/maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteMojo.java
Author: hboutemy
Date: Sat Mar 19 21:27:44 2011
New Revision: 1083296
URL: http://svn.apache.org/viewvc?rev=1083296&view=rev
Log:
extracted getMavenVersion() method
Modified:
maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteMojo.java
Modified: maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteMojo.java?rev=1083296&r1=1083295&r2=1083296&view=diff
==============================================================================
--- maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteMojo.java (original)
+++ maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteMojo.java Sat Mar 19 21:27:44 2011
@@ -20,6 +20,7 @@ package org.apache.maven.plugins.site;
*/
import java.io.File;
+import java.io.IOException;
import java.io.InputStream;
import java.util.List;
@@ -139,32 +140,38 @@ public abstract class AbstractSiteMojo
{
try
{
- // This relies on the fact that MavenProject is the in core classloader
- // and that the core classloader is for the maven-core artifact
- // and that should have a pom.properties file
- // if this ever changes, we will have to revisit this code.
- final Properties properties = new Properties();
- final InputStream in =
- MavenProject.class.getResourceAsStream( "META-INF/maven/org.apache.maven/maven-core/pom.properties" );
- try
- {
- properties.load( in );
- }
- finally
- {
- IOUtil.close( in );
- }
-
- final String version = properties.getProperty( "version" ).trim();
+ final String version = getMavenVersion();
if ( version.startsWith( "3" ) )
{
getLog().warn( "Running 2.x site-plugin with Maven 3, use site-plugin-3.x instead!" );
}
}
- catch ( Exception e )
+ catch ( IOException e )
{
getLog().debug( "Unable to determine Maven version", e );
}
}
+
+ protected String getMavenVersion()
+ throws IOException
+ {
+ // This relies on the fact that MavenProject is the in core classloader
+ // and that the core classloader is for the maven-core artifact
+ // and that should have a pom.properties file
+ // if this ever changes, we will have to revisit this code.
+ final Properties properties = new Properties();
+ final InputStream in =
+ MavenProject.class.getResourceAsStream( "META-INF/maven/org.apache.maven/maven-core/pom.properties" );
+ try
+ {
+ properties.load( in );
+ }
+ finally
+ {
+ IOUtil.close( in );
+ }
+
+ return properties.getProperty( "version" ).trim();
+ }
}