You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2022/12/14 04:29:33 UTC
[doris] 12/13: [chore](macOS) Fix the compilation errors when building third party libraries (#14813)
This is an automated email from the ASF dual-hosted git repository.
morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git
commit 1ad9962e56621342728bf94464cdec0a06c82c1d
Author: Adonis Ling <ad...@gmail.com>
AuthorDate: Wed Dec 7 08:48:06 2022 +0800
[chore](macOS) Fix the compilation errors when building third party libraries (#14813)
---
thirdparty/build-thirdparty.sh | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/thirdparty/build-thirdparty.sh b/thirdparty/build-thirdparty.sh
index 2a6cd85f9f..316f27c9f7 100755
--- a/thirdparty/build-thirdparty.sh
+++ b/thirdparty/build-thirdparty.sh
@@ -147,7 +147,8 @@ elif [[ "${CC}" == *clang ]]; then
boost_toolset='clang'
libhdfs_cxx17='-std=c++1z'
- if "${CC}" -xc++ "${warning_unused_but_set_variable}" /dev/null 2>&1 | grep 'unknown warning option'; then
+ test_warning_result="$("${CC}" -xc++ "${warning_unused_but_set_variable}" /dev/null 2>&1 || true)"
+ if echo "${test_warning_result}" | grep 'unknown warning option' >/dev/null; then
warning_unused_but_set_variable=''
fi
fi
@@ -817,7 +818,7 @@ build_cyrus_sasl() {
check_if_source_exist "${CYRUS_SASL_SOURCE}"
cd "${TP_SOURCE_DIR}/${CYRUS_SASL_SOURCE}"
- CFLAGS="-fPIC" \
+ CFLAGS="-fPIC -Wno-implicit-function-declaration" \
CPPFLAGS="-I${TP_INCLUDE_DIR}" \
LDFLAGS="-L${TP_LIB_DIR}" \
LIBS="-lcrypto" \
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org