Merge pull request #77 from matthew-brett/spinner

Spinner: UI cleanup

common_utils.sh

@@ -18,12 +18,28 @@ if [ $(uname) == "Darwin" ]; then IS_OSX=1; fi
 # https://github.com/direnv/direnv/issues/210
 shell_session_update() { :; }
 
-if [ -n "$MB_KEEP_ALIVE" ]; then
+function start_spinner {
+    if [ -n "$MB_SPINNER_PID" ]; then
+        return
+    fi
+
     # Start a process that runs as a keep-alive
     # to avoid travis quitting if there is no output
+    (while true; do
+        >&2 echo "Building libraries..."
+        sleep 60
+    done) &
+    MB_SPINNER_PID=$!
+}
 
-    (while true; do >&2 echo "Travis-CI keep-alive"; sleep 480; done) &
-fi
+function stop_spinner {
+    if [ ! -n "$MB_SPINNER_PID" ]; then
+        return
+    fi
+    
+    kill $MB_SPINNER_PID
+    unset MB_SPINNER_PID
+}
 
 function abspath {
     python -c "import os.path; print(os.path.abspath('$1'))"
@@ -157,7 +173,11 @@ function build_wheel_cmd {
     local repo_dir=${2:-$REPO_DIR}
     [ -z "$repo_dir" ] && echo "repo_dir not defined" && exit 1
     local wheelhouse=$(abspath ${WHEEL_SDIR:-wheelhouse})
-    if [ -n "$(is_function "pre_build")" ]; then pre_build; fi
+    if [ -n "$(is_function "pre_build")" ]; then 
+        start_spinner
+        pre_build
+        stop_spinner
+    fi
     if [ -n "$BUILD_DEPENDS" ]; then
         pip install $(pip_opts) $BUILD_DEPENDS
     fi

tests/test_library_builders.sh

@@ -8,9 +8,13 @@ source library_builders.sh
 
 if [ "$TRAVIS_OS_NAME" == "osx" ]; then brew update; fi
 
+start_spinner
+
 suppress build_openssl
 suppress build_openblas
 suppress build_libwebp
 suppress build_szip
 suppress build_swig
 suppress build_github fredrik-johansson/arb 2.11.1
+
+stop_spinner