PHILO-HE commented on code in PR #9168: URL: https://github.com/apache/incubator-gluten/pull/9168#discussion_r2018150887
########## ep/build-velox/src/setup-centos7.sh: ########## @@ -233,7 +233,10 @@ dnf_install ccache wget which libevent-devel \ openssl-devel libzstd-devel lz4-devel double-conversion-devel \ curl-devel libxml2-devel libgsasl-devel libuuid-devel patch libicu-devel tzdata -dnf_install https://mirror.stream.centos.org/9-stream/BaseOS/x86_64/os/Packages/tzdata-2025a-1.el9.noarch.rpm +# Update tzdata, required by Velox at runtime. +dnf_install python3-pip +pip3 install tzdata Review Comment: @jinchengchenghh, seems no need to update tzdata on ubuntu? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
