This is an automated email from the ASF dual-hosted git repository.

yuanzhou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new 4dfdde5792 [GLUTEN-10454] Fix CUDF docker build (#10455)
4dfdde5792 is described below

commit 4dfdde579282fe93557a445c25c8191a478b628e
Author: Yuan <[email protected]>
AuthorDate: Fri Aug 15 18:35:05 2025 +0100

    [GLUTEN-10454] Fix CUDF docker build (#10455)
    
    Fix docker image build.
    The issue is in ghcr.io/facebookincubator/velox-dev:adapters, cmake is 
installed under /usr/local/bin.
    With default sudo path not included this.
    
    ---------
    
    Signed-off-by: Yuan <[email protected]>
---
 .github/workflows/docker_image.yml | 2 +-
 dev/build_arrow.sh                 | 6 +++---
 dev/docker/cudf/Dockerfile         | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/docker_image.yml 
b/.github/workflows/docker_image.yml
index be4894bd22..40aefa6c18 100644
--- a/.github/workflows/docker_image.yml
+++ b/.github/workflows/docker_image.yml
@@ -322,7 +322,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        digests: [ vcpkg-centos-8, vcpkg-centos-9,centos-8-jdk8, 
centos-8-jdk11, centos-8-jdk17, centos-9-jdk8, centos-9-jdk11, centos-9-jdk17 ]
+        digests: [ vcpkg-centos-8, vcpkg-centos-9, centos-8-jdk8, 
centos-8-jdk11, centos-8-jdk17, centos-9-jdk8, centos-9-jdk11, centos-9-jdk17 ]
     needs:
       - build-vcpkg-centos-8
       - build-vcpkg-centos-9
diff --git a/dev/build_arrow.sh b/dev/build_arrow.sh
index 30a68bc736..271c1d90c4 100755
--- a/dev/build_arrow.sh
+++ b/dev/build_arrow.sh
@@ -18,14 +18,14 @@
 set -exu
 
 CURRENT_DIR=$(cd "$(dirname "$BASH_SOURCE")"; pwd)
-export SUDO=sudo
+SUDO="${sudo:-""}"
 source ${CURRENT_DIR}/build_helper_functions.sh
 VELOX_ARROW_BUILD_VERSION=15.0.0
 ARROW_PREFIX=$CURRENT_DIR/../ep/_ep/arrow_ep
 BUILD_TYPE=Release
 
 function prepare_arrow_build() {
-  mkdir -p ${ARROW_PREFIX}/../ && pushd ${ARROW_PREFIX}/../ && sudo rm -rf 
arrow_ep/
+  mkdir -p ${ARROW_PREFIX}/../ && pushd ${ARROW_PREFIX}/../ && ${SUDO} rm -rf 
arrow_ep/
   wget_and_untar 
https://github.com/apache/arrow/archive/refs/tags/apache-arrow-${VELOX_ARROW_BUILD_VERSION}/.tar.gz
 arrow_ep
   #wget_and_untar 
https://archive.apache.org/dist/arrow/arrow-${VELOX_ARROW_BUILD_VERSION}/apache-arrow-${VELOX_ARROW_BUILD_VERSION}.tar.gz
 arrow_ep
   cd arrow_ep
@@ -69,7 +69,7 @@ function build_arrow_cpp() {
 
  # Install thrift.
  cd _build/thrift_ep-prefix/src/thrift_ep-build
- sudo cmake --install ./ --prefix /usr/local/
+ ${SUDO} cmake --install ./ --prefix /usr/local/
  popd
 }
 
diff --git a/dev/docker/cudf/Dockerfile b/dev/docker/cudf/Dockerfile
index 32d6faa625..f598f90bf8 100644
--- a/dev/docker/cudf/Dockerfile
+++ b/dev/docker/cudf/Dockerfile
@@ -1,5 +1,5 @@
 FROM ghcr.io/facebookincubator/velox-dev:adapters
-RUN yum install -y sudo patch maven perl
+RUN yum install -y sudo patch maven perl && ln -sf /usr/local/bin/cmake 
/usr/bin
 
 RUN git clone --depth=1 https://github.com/apache/incubator-gluten /opt/gluten
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to