Skip to content

Commit

Permalink
[GLUTEN-7118][VL] Fix duckdb target issue when vcpkg is enabled (#7117)
Browse files Browse the repository at this point in the history
  • Loading branch information
PHILO-HE authored Sep 5, 2024
1 parent 5b7fbbb commit 4f81047
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 3 deletions.
4 changes: 3 additions & 1 deletion dev/package-vcpkg.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ if [ "$LINUX_OS" == "centos" ]; then
if [ "$VERSION" == "8" ]; then
source /opt/rh/gcc-toolset-9/enable
elif [ "$VERSION" == "7" ]; then
export MANPATH=""
source /opt/rh/devtoolset-9/enable
fi
fi
Expand All @@ -22,4 +23,5 @@ source ./dev/vcpkg/env.sh

# build gluten with velox backend, prompt always respond y
export PROMPT_ALWAYS_RESPOND=y
./dev/buildbundle-veloxbe.sh --build_tests=ON --build_benchmarks=ON --enable_s3=ON --enable_hdfs=ON "$@"

./dev/buildbundle-veloxbe.sh --build_tests=ON --build_arrow=OFF --build_benchmarks=ON --enable_s3=ON --enable_hdfs=ON "$@"
2 changes: 1 addition & 1 deletion dev/vcpkg/Makefile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
DOCKER_IMAGE=apache/gluten:gluten-vcpkg-builder
DOCKER_IMAGE=apache/gluten:vcpkg-centos-7
GLUTEN_REPO=$(shell realpath -L ../..)

CCACHE_DIR=$(HOME)/.ccache
Expand Down
7 changes: 7 additions & 0 deletions dev/vcpkg/ports/duckdb/portfile.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,11 @@ vcpkg_configure_cmake(
)
vcpkg_install_cmake(ADD_BIN_TO_PATH)

vcpkg_fixup_cmake_targets(CONFIG_PATH lib/cmake/DuckDB)

file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug/lib/cmake")
file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/lib/cmake")

vcpkg_cmake_config_fixup(PACKAGE_NAME DuckDB)

vcpkg_install_copyright(FILE_LIST "${SOURCE_PATH}/LICENSE")
9 changes: 8 additions & 1 deletion dev/vcpkg/ports/duckdb/vcpkg.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,13 @@
"name": "duckdb",
"version": "0.8.1",
"dependencies": [

{
"name": "vcpkg-cmake",
"host": true
},
{
"name": "vcpkg-cmake-config",
"host": true
}
]
}

0 comments on commit 4f81047

Please sign in to comment.