fixed checkstyle-error by encapsulating PackagerException

FibreFoX · Mar 8, 2016 · 3fa03d7 · 3fa03d7
1 parent f8380e3
commit 3fa03d7
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/main/java/de/dynamicfiles/projects/gradle/plugins/javafx/tasks/JfxNativeTask.java b/src/main/java/de/dynamicfiles/projects/gradle/plugins/javafx/tasks/JfxNativeTask.java
@@ -365,7 +365,7 @@ public FileVisitResult postVisitDirectory(Path source, IOException ioe) throws I
                 // quietly ignored
             } catch(ConfigException e){
                 project.getLogger().info("Skipping '" + b.getName() + "' because of configuration error '" + e.getMessage() + "'\nAdvice to fix: " + e.getAdvice());
-            } catch(PackagerException | GradleException ex){
+            } catch(GradleException ex){
                 throw new GradleException("Got exception while executing bundler.", ex);
             }
         }
@@ -417,7 +417,7 @@ private void applyNativeLauncherWorkaround(Project project, String nativeOutputD
         }
     }
 
-    private void signJarFiles(JavaFXGradlePluginExtension ext) throws PackagerException {
+    private void signJarFiles(JavaFXGradlePluginExtension ext) {
         Project project = this.getProject();
         File keyStore = new File(project.getProjectDir(), ext.getKeyStore());
         if( !keyStore.exists() ){
@@ -456,8 +456,11 @@ private void signJarFiles(JavaFXGradlePluginExtension ext) throws PackagerExcept
         getJARFilesFromJNLPFiles(ext).forEach(jarFile -> signJarParams.addResource(nativeOutputDir, jarFile));
 
         project.getLogger().info("Signing JAR files for webstart bundle");
-
-        new PackagerLib().signJar(signJarParams);
+        try{
+            new PackagerLib().signJar(signJarParams);
+        } catch(PackagerException ex){
+            throw new GradleException("There was a problem while signing JAR files.", ex);
+        }
     }
 
     private List<File> getGeneratedJNLPFiles(JavaFXGradlePluginExtension ext) {