From 7a5d79de9511e3e84aad2adb327b27c9069703ea Mon Sep 17 00:00:00 2001 From: Louis Dionne Date: Wed, 8 Jul 2020 16:27:33 -0400 Subject: [PATCH] [libc++] Make sure the ENABLE_FILESYSTEM variable is always bound The script always fails otherwise, since we run with 'set -u' --- libcxx/utils/ci/macos-backdeployment.sh | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) 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.