Fixes #790 (#798)

Signed-off-by: Scott Kurz <[email protected]> Co-authored-by: Sandy Koh <[email protected]>
appsody · May 1, 2020 · e31befe · e31befe
1 parent c6e1210
commit e31befe
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/incubator/java-openliberty/image/project/run-stack.sh b/incubator/java-openliberty/image/project/run-stack.sh
@@ -24,7 +24,6 @@ case $ACTION in
         else
             set -x
             mvn -B -Plocal-dev -DappsDirectory=apps -Ddebug=false -Dmaven.repo.local=/mvn/repository pre-integration-test liberty:dev
-            set +x
         fi
         ;;
     debug)
@@ -35,7 +34,6 @@ case $ACTION in
         else
             set -x
             mvn -B -Plocal-dev -DappsDirectory=apps -Dmaven.repo.local=/mvn/repository pre-integration-test liberty:dev
-            set +x
         fi
         ;;
     test)
@@ -47,7 +45,7 @@ case $ACTION in
             # Keep liberty:create before 'pre-integration-test' phase to be consistent with "Dockerfile" for 'appsody build'
             set -x
             mvn -B -Plocal-dev -DappsDirectory=apps -Dmaven.repo.local=/mvn/repository clean liberty:create pre-integration-test liberty:install-feature liberty:start liberty:deploy failsafe:integration-test liberty:stop failsafe:verify
-            set +x
+            # make sure exit code propagates
         fi
         ;;
     *)

diff --git a/incubator/java-openliberty/stack.yaml b/incubator/java-openliberty/stack.yaml
@@ -1,5 +1,5 @@
 name: Open Liberty
-version: 0.2.10
+version: 0.2.11
 description: Eclipse MicroProfile & Jakarta EE on Open Liberty & OpenJ9 using Maven
 license: Apache-2.0
 language: java
-Original file line number
+Diff line change
@@ Expand Up / @@ -24,7 +24,6 @@ case $ACTION in @@
             else
                 set -x
                 mvn -B -Plocal-dev -DappsDirectory=apps -Ddebug=false -Dmaven.repo.local=/mvn/repository pre-integration-test liberty:dev
-                set +x
             fi
             ;;
         debug)
@@ Expand All / @@ -35,7 +34,6 @@ case $ACTION in @@
             else
                 set -x
                 mvn -B -Plocal-dev -DappsDirectory=apps -Dmaven.repo.local=/mvn/repository pre-integration-test liberty:dev
-                set +x
             fi
             ;;
         test)
@@ Expand All / @@ -47,7 +45,7 @@ case $ACTION in @@
                 # Keep liberty:create before 'pre-integration-test' phase to be consistent with "Dockerfile" for 'appsody build'
                 set -x
                 mvn -B -Plocal-dev -DappsDirectory=apps -Dmaven.repo.local=/mvn/repository clean liberty:create pre-integration-test liberty:install-feature liberty:start liberty:deploy failsafe:integration-test liberty:stop failsafe:verify
-                set +x
+                # make sure exit code propagates
             fi
             ;;
         *)
@@ Expand Down @@