Skip to content

Commit bab5580

Browse files
authored
GH-47840: [CI][C++] Check whether the CSV module/thread sanitizer is enabled or not before building example (#47841)
### Rationale for this change `cpp/examples/minimal_build/` needs the CSV module. We need additional flags for thread sanitizer. ### What changes are included in this PR? * Check whether `ARROW_CSV` is enabled. * Check whether thread sanitizer is enabled. ### Are these changes tested? Yes. ### Are there any user-facing changes? No. * GitHub Issue: #47840 Authored-by: Sutou Kouhei <[email protected]> Signed-off-by: Raúl Cumplido <[email protected]>
1 parent 7a38744 commit bab5580

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

ci/scripts/cpp_test.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,9 @@ fi
129129
# * This doesn't test other CMake packages such as ArrowDataset
130130
if [ "${ARROW_USE_MESON:-OFF}" = "OFF" ] && \
131131
[ "${ARROW_EMSCRIPTEN:-OFF}" = "OFF" ] && \
132-
[ "${ARROW_USE_ASAN:-OFF}" = "OFF" ]; then
132+
[ "${ARROW_USE_ASAN:-OFF}" = "OFF" ] && \
133+
[ "${ARROW_USE_TSAN:-OFF}" = "OFF" ] && \
134+
[ "${ARROW_CSV:-ON}" = "ON" ]; then
133135
CMAKE_PREFIX_PATH="${CMAKE_INSTALL_PREFIX:-${ARROW_HOME}}"
134136
case "$(uname)" in
135137
MINGW*)

0 commit comments

Comments
 (0)