JENKINS-23908 - created external invoker

Creation of additional type of build step. added docs explaining difference. many classes moved between packages
jenkinsci · Jul 25, 2014 · b4ce408 · b4ce408
1 parent ceb9be7
commit b4ce408
Show file tree

Hide file tree

Showing 17 changed files with 423 additions and 265 deletions.
diff --git a/README.md b/README.md
@@ -1,20 +1,39 @@
-Jenkins Liquibase Plugin
-=================
+# Jenkins Liquibase Plugin
+---
+
+# About
 
-About
------
 Adds Liquibase as an available build step.  See Liquibase documentation at http://www.liquibase.org/.
 
-Installation
-----
+# Installation
 
 *  Install the liquibase-runner plugin.
 *  Create an empty version of your database on the target server (if applicable).
 
 
-Configuration
-----
+# Configuration
+
+## "Execute" versus "Invoke"
+
+There are two ways of using this plugin, distinguished by "Execute Liquibase" versus "Invoke Liquibase"
+
+**Execute**
+
+Use this build step if you have an existing liquibase installation and wish to have it run just as if you were doing so 
+from command line.  The build's console log will contain the stdout/stderr output of liquibase.
+
+**Invoke**
+
+The "Invoke Liquibase" build step runs liquibase internally and doesn't require an existing installation.  Furthermore, 
+any changesets executed during the build are reported.
+
+# The Future
+
+I'd like to minimize the difference between "Invoke" and "Execute" and have changesets be reported in both types of
+build steps.
+
+
+
 
-*  Select "Invoke Liquibase" as the step for any Jenkins Job.
 
 
diff --git a/pom.xml b/pom.xml
@@ -43,7 +43,7 @@
         <dependency>
             <groupId>org.liquibase</groupId>
             <artifactId>liquibase-core</artifactId>
-            <version>3.2.0</version>
+            <version>3.2.2</version>
         </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>

diff --git a/src/main/java/org/jenkinsci/plugins/liquibase/builder/BuildChangeExecListener.java b/src/main/java/org/jenkinsci/plugins/liquibase/builder/BuildChangeExecListener.java
@@ -16,6 +16,7 @@
 import java.util.Arrays;
 import java.util.List;
 
+import org.jenkinsci.plugins.liquibase.common.Util;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 

diff --git a/src/main/java/org/jenkinsci/plugins/liquibase/builder/ChangeSetDetail.java b/src/main/java/org/jenkinsci/plugins/liquibase/builder/ChangeSetDetail.java
@@ -7,6 +7,7 @@
 import java.util.List;
 
 import org.apache.commons.lang.StringUtils;
+import org.jenkinsci.plugins.liquibase.common.Util;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 

diff --git a/src/main/java/org/jenkinsci/plugins/liquibase/builder/Invoker.java b/src/main/java/org/jenkinsci/plugins/liquibase/builder/Invoker.java