You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2006/12/13 07:23:42 UTC

svn commit: r486509 - /geronimo/genesis/trunk/plugins/tools-maven-plugin/src/main/java/org/apache/geronimo/genesis/plugins/tools/VerifyLegalFilesMojo.java

Author: jdillon
Date: Tue Dec 12 22:23:41 2006
New Revision: 486509

URL: http://svn.apache.org/viewvc?view=rev&rev=486509
Log:
Skip artifacts with null files

Modified:
    geronimo/genesis/trunk/plugins/tools-maven-plugin/src/main/java/org/apache/geronimo/genesis/plugins/tools/VerifyLegalFilesMojo.java

Modified: geronimo/genesis/trunk/plugins/tools-maven-plugin/src/main/java/org/apache/geronimo/genesis/plugins/tools/VerifyLegalFilesMojo.java
URL: http://svn.apache.org/viewvc/geronimo/genesis/trunk/plugins/tools-maven-plugin/src/main/java/org/apache/geronimo/genesis/plugins/tools/VerifyLegalFilesMojo.java?view=diff&rev=486509&r1=486508&r2=486509
==============================================================================
--- geronimo/genesis/trunk/plugins/tools-maven-plugin/src/main/java/org/apache/geronimo/genesis/plugins/tools/VerifyLegalFilesMojo.java (original)
+++ geronimo/genesis/trunk/plugins/tools-maven-plugin/src/main/java/org/apache/geronimo/genesis/plugins/tools/VerifyLegalFilesMojo.java Tue Dec 12 22:23:41 2006
@@ -83,6 +83,12 @@
             Artifact artifact = (Artifact)iter.next();
             File file = artifact.getFile();
 
+            // Some artifacts might not have files, so skip them
+            if (file == null) {
+                log.debug("Skipping artifact; no attached file: " + artifact);
+                continue;
+            }
+
             log.info("Verifying legal files: " + file.getName());
             
             try {