[sling-maven-sling-plugin] branch master updated: SLING-7916 - Update documentation for WebDAV bundle deployment

[rombert]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 7542572  SLING-7916 - Update documentation for WebDAV bundle deployment
7542572 is described below

commit 7542572ab079a23962aaff35bb311e9aeb097e98
Author: Robert Munteanu <romb...@apache.org>
AuthorDate: Wed Sep 12 16:38:10 2018 +0200

    SLING-7916 - Update documentation for WebDAV bundle deployment
---
 .../apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java | 2 +-
 .../apache/sling/maven/bundlesupport/AbstractBundlePostMojo.java    | 6 +++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git 
a/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java
 
b/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java
index b8a67c5..c1efd5e 100644
--- 
a/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java
+++ 
b/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundleInstallMojo.java
@@ -47,7 +47,7 @@ abstract class AbstractBundleInstallMojo extends 
AbstractBundlePostMojo {
      *  Make sure that {@link #slingUrl} points to the Felix Web Console in 
that case.</li>
      *  <li><strong>WebDAV</strong>, uses <a 
href="https://sling.apache.org/documentation/development/repository-based-development.html";>
      *  WebDAV</a> for deployment (HTTP PUT). Make sure that {@link #slingUrl} 
points to the entry path of 
-     *  the Sling WebDAV bundle (usually below regular Sling root URL). Issues 
a HTTP Delete for the uninstall goal.
+     *  the Sling WebDAV bundle (defaults to <tt>/dav/default<tt> in the Sling 
starter). Issues a HTTP Delete for the uninstall goal.
      *  <li><strong>SlingPostServlet</strong>, uses the
      *  <a 
href="https://sling.apache.org/documentation/bundles/manipulating-content-the-slingpostservlet-servlets-post.html";>Sling
 Post Servlet</a> for deployment (HTTP POST).
      *  Make sure that {@link #slingUrl} points a path which is handled by the 
Sling POST Servlet (usually below regular Sling root URL).</li>
diff --git 
a/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundlePostMojo.java
 
b/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundlePostMojo.java
index 9f253ab..a9cc085 100644
--- 
a/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundlePostMojo.java
+++ 
b/src/main/java/org/apache/sling/maven/bundlesupport/AbstractBundlePostMojo.java
@@ -34,7 +34,11 @@ import org.apache.maven.plugins.annotations.Parameter;
 abstract class AbstractBundlePostMojo extends AbstractMojo {
 
     /**
-     * The URL of the running Sling instance. The default is only useful for 
<strong>WebConsole</strong> deployment.
+     * The URL of the running Sling instance.
+     *
+     * <p>The default is only useful for <strong>WebConsole</strong> 
deployment.</p>
+     *
+     * <p>For <strong>WebDAV</strong> deployment it is recommended to include 
the Sling Simple WebDAV servlet root, for instance <a 
href="http://localhost:8080/dav/default/libs/sling/install";>http://localhost:8080/dav/default/libs/sling/install</a>.
 Omitting the <tt>dav/default</tt> segment can lead to conflicts with other 
servlets.</p>
      */
     @Parameter(property="sling.url", 
defaultValue="http://localhost:8080/system/console";, required = true)
     protected String slingUrl;