invert wrong logic, skip cfitsio on macOS

multi-build · Jan 27, 2020 · e1d3a27 · e1d3a27
1 parent de94da5
commit e1d3a27
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/common_utils.sh b/common_utils.sh
@@ -432,10 +432,10 @@ function install_pypy {
     "ppc64le") suffix="ppc64le";;
     "s30x")    suffix="s390x";;
     "aarch64")  suffix="aarch64";;
-    *) if [ -z "$IS_OSX" ]; then
+    *) if [ -n "$IS_OSX" ]; then
             suffix = "osx64";
        else
-            echo unknown platform $PLAT; exit 1
+            echo unknown platform "$PLAT"; exit 1
        fi;;
     esac
 

diff --git a/tests/test_library_builders.sh b/tests/test_library_builders.sh
@@ -50,7 +50,10 @@ if [ -z "$IS_OSX" ]; then
     suppress build_lzo
 fi
 suppress build_ragel
-suppress build_cfitsio
+if [ -z "$IS_OSX" ]; then
+    # already installed in the macOS image, so `brew install cfitsio` fails
+    suppress build_cfitsio
+fi
 suppress build_new_zlib
 suppress build_hdf5