This is an automated email from the ASF dual-hosted git repository. chaokunyang pushed a commit to branch releases-0.12 in repository https://gitbox.apache.org/repos/asf/fory.git
commit 9f57d9e064cd407006114d249bea740abe8f1adb Author: Emre Şafak <[email protected]> AuthorDate: Wed Aug 27 01:10:56 2025 -0400 ci: Fix verify_version() by capturing version properly (#2535) ## Why? [Containerized builds were failing at the version verification step](https://github.com/apache/fory/actions/runs/17257282639/job/48971637440#step:5:1997). ## What does this PR do? * Refine script to capture only the actual version output from deploy.sh. --- ci/deploy.sh | 4 +++- ci/tasks/python_container_build_script.sh | 4 ++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/ci/deploy.sh b/ci/deploy.sh index 5ec402e05..cb9a23cf7 100755 --- a/ci/deploy.sh +++ b/ci/deploy.sh @@ -19,7 +19,9 @@ # Print commands and their arguments as they are executed. -set -x +if [ "${DEPLOY_QUIET:-0}" != "1" ]; then + set -x +fi # Cause the script to exit if a single command fails. set -e diff --git a/ci/tasks/python_container_build_script.sh b/ci/tasks/python_container_build_script.sh index c89e961f9..da4032d6e 100644 --- a/ci/tasks/python_container_build_script.sh +++ b/ci/tasks/python_container_build_script.sh @@ -32,9 +32,9 @@ verify_version() { # Check if GITHUB_REF_NAME is available and use it for verification if [ -n "$GITHUB_REF_NAME" ]; then - # Strip leading 'v' if present + # Strip leading 'v' if present and capture only the actual output, not debug messages local expected_version - expected_version="$(ci/deploy.sh parse_py_version $GITHUB_REF_NAME)" + expected_version="$(DEPLOY_QUIET=1 ci/deploy.sh parse_py_version $GITHUB_REF_NAME)" echo "Expected version: $expected_version" if [ "$installed_version" != "$expected_version" ]; then --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
