[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