raulcd commented on code in PR #48223:
URL: https://github.com/apache/arrow/pull/48223#discussion_r2564488921
##########
ci/scripts/cpp_build.sh:
##########
@@ -141,164 +144,166 @@ if [ "${ARROW_USE_MESON:-OFF}" = "ON" ]; then
fi
fi
meson setup \
- --prefix=${MESON_PREFIX:-${ARROW_HOME}} \
- --buildtype=${ARROW_BUILD_TYPE:-debug} \
+ --prefix="${MESON_PREFIX:-${ARROW_HOME}}" \
+ --buildtype="${ARROW_BUILD_TYPE:-debug}" \
--pkg-config-path="${CONDA_PREFIX}/lib/pkgconfig/" \
-Dauto_features=enabled \
-Dfuzzing=disabled \
-Ds3=disabled \
. \
- ${source_dir}
+ "${source_dir}"
CC="${ORIGINAL_CC}"
CXX="${ORIGINAL_CXX}"
elif [ "${ARROW_EMSCRIPTEN:-OFF}" = "ON" ]; then
if [ "${UBUNTU}" = "20.04" ]; then
echo "arrow emscripten build is not supported on Ubuntu 20.04, run with
UBUNTU=22.04"
- exit -1
+ exit 1
fi
n_jobs=2 # Emscripten build fails on docker unless this is set really low
+ # shellcheck source=/dev/null
source ~/emsdk/emsdk_env.sh
- export CMAKE_INSTALL_PREFIX=$(em-config CACHE)/sysroot
+ CMAKE_INSTALL_PREFIX=$(em-config CACHE)/sysroot
+ export CMAKE_INSTALL_PREFIX
# conda sets LDFLAGS / CFLAGS etc. which break
# emcmake so we unset them
unset LDFLAGS CFLAGS CXXFLAGS CPPFLAGS
emcmake cmake \
- --preset=ninja-${ARROW_BUILD_TYPE:-debug}-emscripten \
- -DCMAKE_VERBOSE_MAKEFILE=${CMAKE_VERBOSE_MAKEFILE:-OFF} \
+ --preset=ninja-"${ARROW_BUILD_TYPE:-debug}"-emscripten \
+ -DCMAKE_VERBOSE_MAKEFILE="${CMAKE_VERBOSE_MAKEFILE:-OFF}" \
-DCMAKE_C_FLAGS="${CFLAGS:-}" \
-DCMAKE_CXX_FLAGS="${CXXFLAGS:-}" \
-DCMAKE_CXX_STANDARD="${CMAKE_CXX_STANDARD:-17}" \
- -DCMAKE_INSTALL_LIBDIR=${CMAKE_INSTALL_LIBDIR:-lib} \
- -DCMAKE_INSTALL_PREFIX=${CMAKE_INSTALL_PREFIX:-${ARROW_HOME}} \
- -DCMAKE_UNITY_BUILD=${CMAKE_UNITY_BUILD:-OFF} \
- ${ARROW_CMAKE_ARGS} \
- ${source_dir}
+ -DCMAKE_INSTALL_LIBDIR="${CMAKE_INSTALL_LIBDIR:-lib}" \
+ -DCMAKE_INSTALL_PREFIX="${CMAKE_INSTALL_PREFIX:-${ARROW_HOME}}" \
+ -DCMAKE_UNITY_BUILD="${CMAKE_UNITY_BUILD:-OFF}" \
+ "${ARROW_CMAKE_ARGS}" \
Review Comment:
The MinGW failures seem related to this expansion:
> cmake -Dabsl_SOURCE= -DARROW_ACERO=OFF -DARROW_AZURE=OFF
-DARROW_BOOST_USE_SHARED=ON -DARROW_BUILD_BENCHMARKS_REFERENCE=OFF
-DARROW_BUILD_BENCHMARKS=OFF -DARROW_BUILD_EXAMPLES=OFF
-DARROW_BUILD_INTEGRATION=OFF -DARROW_BUILD_SHARED=ON -DARROW_BUILD_STATIC=OFF
-DARROW_BUILD_TESTS=ON -DARROW_BUILD_UTILITIES=ON -DARROW_COMPUTE=ON
-DARROW_CSV=ON -DARROW_CUDA=OFF -DARROW_CXXFLAGS= -DARROW_CXX_FLAGS_DEBUG=-g1
-DARROW_CXX_FLAGS_RELEASE= -DARROW_CXX_FLAGS_RELWITHDEBINFO=
-DARROW_C_FLAGS_DEBUG=-g1 -DARROW_C_FLAGS_RELEASE=
-DARROW_C_FLAGS_RELWITHDEBINFO= -DARROW_DATASET=ON
-DARROW_DEPENDENCY_SOURCE=AUTO -DARROW_DEPENDENCY_USE_SHARED=ON
-DARROW_ENABLE_THREADING=ON -DARROW_ENABLE_TIMING_TESTS=OFF
-DARROW_EXTRA_ERROR_CONTEXT=OFF -DARROW_FILESYSTEM=ON -DARROW_FLIGHT=ON
-DARROW_FLIGHT_SQL=ON -DARROW_FLIGHT_SQL_ODBC=OFF -DARROW_FUZZING=OFF
-DARROW_GANDIVA_PC_CXX_FLAGS= -DARROW_GANDIVA=ON -DARROW_GCS=ON
-DARROW_HDFS=OFF -DARROW_INSTALL_NAME_RPATH=ON -DARROW_JEMALLOC=OFF
-DARROW_JSON=ON -DARROW_LA
RGE_MEMORY_TESTS=OFF -DARROW_MIMALLOC=ON -DARROW_ORC=OFF -DARROW_PARQUET=ON
-DARROW_RUNTIME_SIMD_LEVEL=MAX -DARROW_S3=ON -DARROW_SIMD_LEVEL=DEFAULT
-DARROW_SUBSTRAIT=ON -DARROW_TEST_LINKAGE=shared -DARROW_TEST_MEMCHECK=OFF
-DARROW_USE_ASAN=OFF -DARROW_USE_CCACHE=ON -DARROW_USE_GLOG=OFF
-DARROW_USE_LLD=OFF -DARROW_USE_MOLD=OFF -DARROW_USE_STATIC_CRT=OFF
-DARROW_USE_TSAN=OFF -DARROW_USE_UBSAN=OFF -DARROW_VERBOSE_THIRDPARTY_BUILD=OFF
-DARROW_WITH_BROTLI=ON -DARROW_WITH_BZ2=ON -DARROW_WITH_LZ4=ON
-DARROW_WITH_OPENTELEMETRY=OFF -DARROW_WITH_MUSL=OFF -DARROW_WITH_SNAPPY=ON
-DARROW_WITH_UTF8PROC=ON -DARROW_WITH_ZLIB=ON -DARROW_WITH_ZSTD=ON
-DAWSSDK_SOURCE= -DAzure_SOURCE= -Dbenchmark_SOURCE= -DBOOST_SOURCE=
-DBrotli_SOURCE= -DBUILD_WARNING_LEVEL=CHECKIN -Dc-ares_SOURCE=
-DCMAKE_BUILD_TYPE=release -DCMAKE_VERBOSE_MAKEFILE=OFF -DCMAKE_C_FLAGS=
-DCMAKE_CXX_FLAGS= -DCMAKE_CXX_STANDARD=17 -DCMAKE_INSTALL_LIBDIR=lib
-DCMAKE_INSTALL_PREFIX=/mingw64 -DCMAKE_UNITY_BUILD=OFF -DCUDAToolkit_ROOT= -Dgf
lags_SOURCE= -Dgoogle_cloud_cpp_storage_SOURCE= -DgRPC_SOURCE= -DGTest_SOURCE=
-Dlz4_SOURCE= -Dopentelemetry-cpp_SOURCE= -DORC_SOURCE=
-DPARQUET_BUILD_EXAMPLES=OFF -DPARQUET_BUILD_EXECUTABLES=OFF
-DPARQUET_REQUIRE_ENCRYPTION=ON -DProtobuf_SOURCE= -DRapidJSON_SOURCE=
-Dre2_SOURCE= -DSnappy_SOURCE= -DThrift_SOURCE= -Dutf8proc_SOURCE=
-Dzstd_SOURCE= -Dxsimd_SOURCE= -G Ninja '-DARROW_PACKAGE_PREFIX=/mingw64
-DCMAKE_FIND_PACKAGE_PREFER_CONFIG=ON' /d/a/arrow/arrow/cpp
See:
https://github.com/apache/arrow/blob/39c865e5d65d00cc1e85cf16847ed5b348b55719/.github/workflows/cpp.yml#L331-L333
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
