Skip to content

Commit 1d63348

Browse files
authored
[VL] Fix setup-rhel script (apache#12334)
Need to use fbthrift installed from velox build
1 parent e2300d8 commit 1d63348

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

ep/build-velox/src/setup-rhel.sh

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -131,9 +131,11 @@ function install_elfutils-libelf {
131131

132132
function install_xxHash {
133133
wget_and_untar https://github.com/Cyan4973/xxHash/archive/refs/tags/v${XXHASH_VERSION}.tar.gz xxHash
134-
cd ${DEPENDENCY_DIR}/xxHash
135-
make "-j${NPROC}"
136-
make install PREFIX=${INSTALL_PREFIX}
134+
(
135+
cd ${DEPENDENCY_DIR}/xxHash
136+
make "-j${NPROC}"
137+
make install PREFIX=${INSTALL_PREFIX}
138+
)
137139
}
138140

139141
function install_googletest {
@@ -252,7 +254,6 @@ function install_velox_deps {
252254
run_and_time install_fbthrift
253255
run_and_time install_duckdb
254256
run_and_time install_stemmer
255-
run_and_time install_thrift
256257
run_and_time install_simdjson
257258
run_and_time install_geos
258259
}

0 commit comments

Comments
 (0)