Merge pull request #87 from seiko2plus/fix_strip

Fix stripping binaries from debugging symbols

Author: charris

env_vars.sh

@@ -1,5 +1,15 @@
+#!/usr/bin/env bash
 # Environment variables for build
 OPENBLAS_VERSION="v0.3.7"
 MACOSX_DEPLOYMENT_TARGET=10.9
 CFLAGS="-std=c99 -fno-strict-aliasing"
-LDFLAGS="-Wl,--strip-debug"
+# Macos's linker doesn't support stripping symbols
+if [ "$(uname)" != "Darwin" ]; then
+    LDFLAGS="-Wl,--strip-debug"
+    # make sure that LDFLAGS is exposed to child processes,
+    # since current version of manybuild only export CFLAGS, CPPFLAGS and FFLAGS
+    export LDFLAGS="$LDFLAGS"
+    # override the default stripping flags, since we only override CFLAGS and
+    # the current version of manybuild pass "-strip-all" to CPPFLAGS and FFLAGS
+    STRIP_FLAGS=""
+fi

env_vars_32.sh

@@ -1,3 +1,4 @@
+#!/usr/bin/env bash
 # Environment variables for 32-bit build.
 # The important difference from the 64-bit build is `-msse2` to
 # compile sse loops for ufuncs.
@@ -7,4 +8,13 @@ MACOSX_DEPLOYMENT_TARGET=10.9
 # Causes failure for pre-1.19 in np.reciprocal
 # CFLAGS="-msse2 -std=c99 -fno-strict-aliasing"
 CFLAGS="-std=c99 -fno-strict-aliasing"
-LDFLAGS="-Wl,--strip-debug"
+# Macos's linker doesn't support stripping symbols
+if [ "$(uname)" != "Darwin" ]; then
+    LDFLAGS="-Wl,--strip-debug"
+    # make sure that LDFLAGS is exposed to child processes,
+    # since current version of manybuild only export CFLAGS, CPPFLAGS and FFLAGS
+    export LDFLAGS="$LDFLAGS"
+    # override the default stripping flags, since we only override CFLAGS and
+    # the current version of manybuild pass "-strip-all" to CPPFLAGS and FFLAGS
+    STRIP_FLAGS=""
+fi