You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ro...@apache.org on 2016/11/14 12:21:30 UTC
svn commit: r1769613 -
/sling/trunk/tooling/maven/slingstart-maven-plugin/src/main/java/org/apache/sling/maven/slingstart/PreparePackageMojo.java
Author: rombert
Date: Mon Nov 14 12:21:29 2016
New Revision: 1769613
URL: http://svn.apache.org/viewvc?rev=1769613&view=rev
Log:
SLING-6284 - PreparePackageMojo.testSubsystemBaseGeneration often fails
on Jenkins
Fail early if the mkdirs() call fails.
Modified:
sling/trunk/tooling/maven/slingstart-maven-plugin/src/main/java/org/apache/sling/maven/slingstart/PreparePackageMojo.java
Modified: sling/trunk/tooling/maven/slingstart-maven-plugin/src/main/java/org/apache/sling/maven/slingstart/PreparePackageMojo.java
URL: http://svn.apache.org/viewvc/sling/trunk/tooling/maven/slingstart-maven-plugin/src/main/java/org/apache/sling/maven/slingstart/PreparePackageMojo.java?rev=1769613&r1=1769612&r2=1769613&view=diff
==============================================================================
--- sling/trunk/tooling/maven/slingstart-maven-plugin/src/main/java/org/apache/sling/maven/slingstart/PreparePackageMojo.java (original)
+++ sling/trunk/tooling/maven/slingstart-maven-plugin/src/main/java/org/apache/sling/maven/slingstart/PreparePackageMojo.java Mon Nov 14 12:21:29 2016
@@ -330,7 +330,10 @@ public class PreparePackageMojo extends
Manifest runModesManifest = getRunModesManifest(feature);
getLog().info("Creating subsystem base file: " + subsystemFile.getName());
- subsystemFile.getParentFile().mkdirs();
+ boolean created = subsystemFile.getParentFile().mkdirs();
+ if ( !created ) {
+ throw new MojoExecutionException("Failed creating " + subsystemFile.getParentFile().getAbsolutePath());
+ }
try (JarOutputStream os = new JarOutputStream(new FileOutputStream(subsystemFile), runModesManifest)) {
Map<String, Integer> bsnStartOrderMap = new HashMap<>();