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

philo 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 369ae16371 [MINOR] Avoid duplicate comment symbol in setup scripts 
(#8360)
369ae16371 is described below

commit 369ae16371310bc505c29e5619e9f5c9074ed4f5
Author: Zouxxyy <[email protected]>
AuthorDate: Fri Dec 27 16:28:33 2024 +0800

    [MINOR] Avoid duplicate comment symbol in setup scripts (#8360)
---
 ep/build-velox/src/get_velox.sh | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/ep/build-velox/src/get_velox.sh b/ep/build-velox/src/get_velox.sh
index a6808e70ee..cb1e76aa5d 100755
--- a/ep/build-velox/src/get_velox.sh
+++ b/ep/build-velox/src/get_velox.sh
@@ -118,19 +118,19 @@ function process_setup_centos9 {
 }
 
 function process_setup_alinux3 {
-  sed -i "s/.*dnf_install epel-release/#&/" ${CURRENT_DIR}/setup-centos8.sh
-  sed -i "s/.*run_and_time install_conda/#&/" ${CURRENT_DIR}/setup-centos8.sh
-  sed -i "s/.*dnf config-manager --set-enabled powertools/#&/" 
${CURRENT_DIR}/setup-centos8.sh
+  sed -i "/^[[:space:]]*#/!s/.*dnf_install epel-release/#&/" 
${CURRENT_DIR}/setup-centos8.sh
+  sed -i "/^[[:space:]]*#/!s/.*run_and_time install_conda/#&/" 
${CURRENT_DIR}/setup-centos8.sh
+  sed -i "/^[[:space:]]*#/!s/.*dnf config-manager --set-enabled 
powertools/#&/" ${CURRENT_DIR}/setup-centos8.sh
   sed -i "s/gcc-toolset-11 //" ${CURRENT_DIR}/setup-centos8.sh
-  sed -i "s/.*source \/opt\/rh\/gcc-toolset-11\/enable/#&/" 
${CURRENT_DIR}/setup-centos8.sh
-  sed -i 's|^export CC=/opt/rh/gcc-toolset-11/root/bin/gcc|# &|' 
${CURRENT_DIR}/setup-centos8.sh
-  sed -i 's|^export CXX=/opt/rh/gcc-toolset-11/root/bin/g++|# &|' 
${CURRENT_DIR}/setup-centos8.sh
+  sed -i "/^[[:space:]]*#/!s/.*source \/opt\/rh\/gcc-toolset-11\/enable/#&/" 
${CURRENT_DIR}/setup-centos8.sh
+  sed -i "/^[[:space:]]*#/!s|^export CC=/opt/rh/gcc-toolset-11/root/bin/gcc|# 
&|" ${CURRENT_DIR}/setup-centos8.sh
+  sed -i "/^[[:space:]]*#/!s|^export CXX=/opt/rh/gcc-toolset-11/root/bin/g++|# 
&|" ${CURRENT_DIR}/setup-centos8.sh
   sed -i 's/python39 python39-devel python39-pip //g' 
${CURRENT_DIR}/setup-centos8.sh
-  sed -i "s/.*pip.* install/#&/" ${CURRENT_DIR}/setup-centos8.sh
+  sed -i "/^[[:space:]]*#/!s/.*pip.* install/#&/" 
${CURRENT_DIR}/setup-centos8.sh
 }
 
 function process_setup_tencentos32 {
-  sed -i "s/.*dnf config-manager --set-enabled powertools/#&/" 
${CURRENT_DIR}/setup-centos8.sh
+  sed -i "/^[[:space:]]*#/!s/.*dnf config-manager --set-enabled 
powertools/#&/" ${CURRENT_DIR}/setup-centos8.sh
 }
 
 echo "Preparing Velox source code..."


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

Reply via email to