You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by fm...@apache.org on 2007/09/24 10:38:10 UTC
svn commit: r578708 -
/incubator/sling/trunk/maven-sling-plugin/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java
Author: fmeschbe
Date: Mon Sep 24 01:38:09 2007
New Revision: 578708
URL: http://svn.apache.org/viewvc?rev=578708&view=rev
Log:
Fail project build if installation fails
Modified:
incubator/sling/trunk/maven-sling-plugin/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java
Modified: incubator/sling/trunk/maven-sling-plugin/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java
URL: http://svn.apache.org/viewvc/incubator/sling/trunk/maven-sling-plugin/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java?rev=578708&r1=578707&r2=578708&view=diff
==============================================================================
--- incubator/sling/trunk/maven-sling-plugin/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java (original)
+++ incubator/sling/trunk/maven-sling-plugin/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java Mon Sep 24 01:38:09 2007
@@ -98,7 +98,7 @@
post(slingUrl, bundleFile);
}
- protected void post(String targetURL, File file) {
+ protected void post(String targetURL, File file) throws MojoExecutionException {
// append pseudo path after root URL to not get redirected for nothing
PostMethod filePost = new PostMethod(targetURL + "/install");
@@ -135,12 +135,8 @@
getLog().error(
"Installation failed, cause: " + HttpStatus.getStatusText(status));
}
- } catch (ConnectException ce) {
- getLog().info("Installation on " + targetURL + " failed, cause: " + ce.getMessage());
- getLog().debug(ce); // dump on debug
} catch (Exception ex) {
- getLog().error(ex.getClass().getName() + " " + ex.getMessage());
- ex.printStackTrace();
+ throw new MojoExecutionException("Installation on " + targetURL + " failed, cause: " + ex.getMessage(), ex);
} finally {
filePost.releaseConnection();
}