svn commit: r416513 - /maven/plugins/trunk/maven-jar-plugin/src/main/java/org/apache/maven/plugin/jar/JarSignMojo.java

[mp...@apache.org]

Author: mperham
Date: Thu Jun 22 16:39:28 2006
New Revision: 416513

URL: http://svn.apache.org/viewvc?rev=416513&view=rev
Log:
PR: MJAR-27
Submitted by: Michael Böckling
Skip jar:sign for modules which are not java modules

Modified:
    maven/plugins/trunk/maven-jar-plugin/src/main/java/org/apache/maven/plugin/jar/JarSignMojo.java

Modified: maven/plugins/trunk/maven-jar-plugin/src/main/java/org/apache/maven/plugin/jar/JarSignMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-jar-plugin/src/main/java/org/apache/maven/plugin/jar/JarSignMojo.java?rev=416513&r1=416512&r2=416513&view=diff
==============================================================================
--- maven/plugins/trunk/maven-jar-plugin/src/main/java/org/apache/maven/plugin/jar/JarSignMojo.java (original)
+++ maven/plugins/trunk/maven-jar-plugin/src/main/java/org/apache/maven/plugin/jar/JarSignMojo.java Thu Jun 22 16:39:28 2006
@@ -22,6 +22,7 @@
 import org.apache.maven.plugin.logging.Log;
 import org.apache.maven.project.MavenProject;
 import org.apache.maven.project.MavenProjectHelper;
+import org.apache.maven.artifact.handler.ArtifactHandler;
 import org.codehaus.plexus.util.StringUtils;
 import org.codehaus.plexus.util.cli.CommandLineException;
 import org.codehaus.plexus.util.cli.CommandLineUtils;
@@ -184,6 +185,16 @@
         if ( skip )
         {
             getLog().info( "Skipping JAR signing for file: " + getJarFile().getAbsolutePath() );
+        }
+
+        if ( project != null )
+        {
+            ArtifactHandler artifactHandler = project.getArtifact().getArtifactHandler();
+            if ( artifactHandler != null && !"java".equals( artifactHandler.getLanguage() ) )
+            {
+                getLog().debug( "Not executing jar:sign as the project is not a Java module" );
+                return;
+            }
         }
 
         // we use this mojo to check if there's a need to sign.