Skip to content

Commit 2437992

Browse files
authored
Switch to libstdc++ as default (#6775)
1 parent 32c4cdd commit 2437992

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed

.github/workflows/long-test.yml

+3-2
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ jobs:
5353
git config --global --add safe.directory "$GITHUB_WORKSPACE"
5454
mkdir build
5555
cd build
56-
cmake -GNinja -DCOMPILE_TARGET=virtual -DCMAKE_BUILD_TYPE=Debug -DLONG_TESTS=ON -DSAN=ON -DUSE_LIBCXX=OFF ..
56+
cmake -GNinja -DCOMPILE_TARGET=virtual -DCMAKE_BUILD_TYPE=Debug -DLONG_TESTS=ON -DSAN=ON ..
5757
ninja
5858
5959
- name: "Test"
@@ -177,7 +177,8 @@ jobs:
177177
git config --global --add safe.directory /__w/CCF/CCF
178178
mkdir build
179179
cd build
180-
cmake -GNinja -DCOMPILE_TARGET=virtual -DCMAKE_BUILD_TYPE=Debug -DLONG_TESTS=ON ..
180+
# Use libc++ to enable hardening/bounds checking during tests
181+
cmake -GNinja -DCOMPILE_TARGET=virtual -DCMAKE_BUILD_TYPE=Debug -DLONG_TESTS=ON -DUSE_LIBCXX=ON ..
181182
ninja
182183
183184
- name: "Test"

CMakeLists.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ endif()
113113
option(USE_SNMALLOC "Link virtual build against snmalloc" ON)
114114

115115
# Default inherited from Open Enclave usage
116-
option(USE_LIBCXX "Use libc++ instead of libstdc++" ON)
116+
option(USE_LIBCXX "Use libc++ instead of libstdc++" OFF)
117117

118118
enable_language(ASM)
119119

cmake/cpack_settings.cmake

+3-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,9 @@ set(CCF_DEB_BASE_DEPENDENCIES
2626
)
2727
set(CCF_DEB_DEPENDENCIES ${CCF_DEB_BASE_DEPENDENCIES})
2828

29-
list(APPEND CCF_DEB_DEPENDENCIES "libc++1-15;libc++abi1-15")
29+
if(USE_LIBCXX)
30+
list(APPEND CCF_DEB_DEPENDENCIES "libc++1-15;libc++abi1-15")
31+
endif()
3032

3133
list(JOIN CCF_DEB_DEPENDENCIES ", " CPACK_DEBIAN_PACKAGE_DEPENDS)
3234

0 commit comments

Comments
 (0)