Make sure tests displays errors on windows, fix some bad practise in bash

appveyor.yml

@@ -12,5 +12,4 @@ before_build:
 
 build_script:    
   - C:\msys64\usr\bin\bash -lc "script=compile C:/projects/jucipp/ci/execute.sh"
-  - C:\msys64\usr\bin\bash -lc "broadwayd & exit"
+  - C:\msys64\usr\bin\bash -lc "script=compile make_command='CTEST_OUTPUT_ON_FAILURE=1 make test' C:/projects/jucipp/ci/execute.sh"
-  - C:\msys64\usr\bin\bash -lc "script=compile make_command='make test' C:/projects/jucipp/ci/execute.sh"

ci/compile.sh

@@ -8,6 +8,7 @@ if [ "${cmake_command}" == "" ]; then
       mingw="mingw32"
     fi
     cmake_command="cmake -G\"MSYS Makefiles\" -DCMAKE_INSTALL_PREFIX=/${mingw} -DENABLE_TESTING=1 -DCMAKE_CXX_FLAGS=-Werror .."
+    make_command="make"
   else
     cmake_command="cmake -DENABLE_TESTING=1 -DCMAKE_CXX_FLAGS=-Werror .."
   fi
@@ -17,8 +18,8 @@ if [ "${make_command}" == "" ]; then
   make_command="make -j 2"
 fi
 
-cd jucipp || (echo "Can't cd into jucipp"; return 1)
+cd jucipp || echo "Can't cd into jucipp"
-git submodule update --init --recursive
+git submodule update --init --recursive # appveyor doesn't checkout recursively
-mkdir -p build && cd build || (echo "Error making build directory"; return 1)
+mkdir -p build && cd build || echo "Error making build directory"
-sh -c "${cmake_command}" || (echo "Cmake configuration failed"; return 1)
+sh -c "${cmake_command}" || echo "Cmake configuration failed"
 exec sh -c "${make_command}"