From 70672648fbd0837e94dbc50b4d1aeaa343236117 Mon Sep 17 00:00:00 2001 From: Jonas Hahnfeld Date: Thu, 31 Oct 2024 14:13:53 +0100 Subject: [PATCH] builtin_davix: Drop PATCH_COMMAND The build fix for FreeBSD is included already since DAVIX 0.8.6, so the PATCH_COMMAND is not needed (it was merged because the PR was older, opened in February, than the upgrade of the builtin in May). In fact the command breaks builtin_davix on Linux systems because `sed -i` is not portable between GNU sed and BSD sed. (Note that the same is true for the patch itself, see my comment in https://github.com/cern-fts/davix/pull/113#issuecomment-2449788349 .) --- builtins/davix/CMakeLists.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/builtins/davix/CMakeLists.txt b/builtins/davix/CMakeLists.txt index a78e7156c3881..7ac84367df0e7 100644 --- a/builtins/davix/CMakeLists.txt +++ b/builtins/davix/CMakeLists.txt @@ -48,7 +48,6 @@ ExternalProject_Add(DAVIX -DCMAKE_OSX_SYSROOT=${CMAKE_OSX_SYSROOT} -DCMAKE_OSX_DEPLOYMENT_TARGET=${CMAKE_OSX_DEPLOYMENT_TARGET} -DLIB_SUFFIX= - PATCH_COMMAND sed -i "" -e "s|sed -i '| sed -i \"\" '|g" -e "s/gcc/cc/g" patch-curl-clock-gettime.sh LOG_BUILD 1 LOG_CONFIGURE 1 LOG_DOWNLOAD 1 LOG_INSTALL 1 BUILD_BYPRODUCTS ${DAVIX_LIBRARIES} TIMEOUT 600)