You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2019/10/16 11:21:20 UTC

[sling-slingfeature-maven-plugin] branch master updated: SLING-8779 : NPE in apis mojo when generating javadoc

This is an automated email from the ASF dual-hosted git repository.

cziegeler pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-slingfeature-maven-plugin.git


The following commit(s) were added to refs/heads/master by this push:
     new 2e42d96  SLING-8779 : NPE in apis mojo when generating javadoc
2e42d96 is described below

commit 2e42d967d394c53c88d22d7ac06b0e7712662b27
Author: Carsten Ziegeler <cz...@apache.org>
AuthorDate: Wed Oct 16 13:20:39 2019 +0200

    SLING-8779 : NPE in apis mojo when generating javadoc
---
 .../java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java    | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java b/src/main/java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java
index c88dd50..2e7d9dd 100644
--- a/src/main/java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java
+++ b/src/main/java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java
@@ -509,8 +509,12 @@ public class ApisJarMojo extends AbstractIncludingFeatureMojo implements Artifac
         }
 
         for (org.apache.maven.artifact.Artifact resolvedArtifact : result.getArtifacts()) {
-            getLog().debug("Adding to javadoc classpath " + resolvedArtifact);
-            javadocClasspath.add(resolvedArtifact.getFile().getAbsolutePath());
+            if (resolvedArtifact.getFile() != null) {
+                getLog().debug("Adding to javadoc classpath " + resolvedArtifact);
+                javadocClasspath.add(resolvedArtifact.getFile().getAbsolutePath());
+            } else {
+                getLog().debug("Ignoring for javadoc classpath " + resolvedArtifact);
+            }
         }
     }