[libcxx-commits] [libcxx] 7a5d79d - [libc++] Make sure the ENABLE_FILESYSTEM variable is always bound

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Wed Jul 8 13:28:46 PDT 2020


Author: Louis Dionne
Date: 2020-07-08T16:28:33-04:00
New Revision: 7a5d79de9511e3e84aad2adb327b27c9069703ea

URL: https://github.com/llvm/llvm-project/commit/7a5d79de9511e3e84aad2adb327b27c9069703ea
DIFF: https://github.com/llvm/llvm-project/commit/7a5d79de9511e3e84aad2adb327b27c9069703ea.diff

LOG: [libc++] Make sure the ENABLE_FILESYSTEM variable is always bound

The script always fails otherwise, since we run with 'set -u'

Added: 
    

Modified: 
    libcxx/utils/ci/macos-backdeployment.sh

Removed: 
    


################################################################################
diff  --git a/libcxx/utils/ci/macos-backdeployment.sh b/libcxx/utils/ci/macos-backdeployment.sh
index 839c61386824..d86337e4db37 100755
--- a/libcxx/utils/ci/macos-backdeployment.sh
+++ b/libcxx/utils/ci/macos-backdeployment.sh
@@ -120,8 +120,10 @@ LIBCXX_ROOT_ON_DEPLOYMENT_TARGET="${PREVIOUS_DYLIBS_DIR}/macOS/libc++/${DEPLOYME
 LIBCXXABI_ROOT_ON_DEPLOYMENT_TARGET="${PREVIOUS_DYLIBS_DIR}/macOS/libc++abi/${DEPLOYMENT_TARGET}"
 
 # Filesystem is supported on Apple platforms starting with macosx10.15.
-if [[ ${DEPLOYMENT_TARGET} =~ "^10.9|10.10|10.11|10.12|10.13|10.14$" ]]; then
-    ENABLE_FILESYSTEM="--param enable_filesystem=False"
+if [[ ${DEPLOYMENT_TARGET} =~ ^10.9|10.10|10.11|10.12|10.13|10.14$ ]]; then
+    ENABLE_FILESYSTEM="--param enable_filesystem=false"
+else
+    ENABLE_FILESYSTEM="--param enable_filesystem=true"
 fi
 
 # TODO: We need to also run the tests for libc++abi.


        


More information about the libcxx-commits mailing list