You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by de...@apache.org on 2007/11/04 22:23:37 UTC

svn commit: r591839 - /maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteJarMojo.java

Author: dennisl
Date: Sun Nov  4 13:23:36 2007
New Revision: 591839

URL: http://svn.apache.org/viewvc?rev=591839&view=rev
Log:
o Minor refactoring to prepare for other archivers.

Modified:
    maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteJarMojo.java

Modified: maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteJarMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteJarMojo.java?rev=591839&r1=591838&r2=591839&view=diff
==============================================================================
--- maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteJarMojo.java (original)
+++ maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteJarMojo.java Sun Nov  4 13:23:36 2007
@@ -84,11 +84,12 @@
 
         try
         {
-            File outputFile = createArchive( outputDirectory, finalName + "-site.jar" );
+            File outputFile = createArchive( outputDirectory,
+                                             finalName + "-" + getClassifier() + "." + getArtifactType() );
 
             if ( attach )
             {
-                projectHelper.attachArtifact( project, "jar", "site", outputFile );
+                projectHelper.attachArtifact( project, getArtifactType(), getClassifier(), outputFile );
             }
             else
             {
@@ -103,6 +104,16 @@
         {
             throw new MojoExecutionException( "Error while creating archive.", e );
         }
+    }
+
+    protected String getArtifactType()
+    {
+        return "jar";
+    }
+
+    protected String getClassifier()
+    {
+        return "site";
     }
 
     /**