Skip to content

Commit

Permalink
Fix verification script, verify_rc.sh (#199)
Browse files Browse the repository at this point in the history
* use `dist/release`

* local changes

* fix rc.yml

* use downloads

* fix url
  • Loading branch information
kevinjqliu authored Nov 13, 2024
1 parent ac8fd58 commit 8ca0bb7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/rc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ jobs:
echo "RC=${rc}" >> ${GITHUB_ENV}
- name: Archive
run: |
id="apache-iceberg-go-${VERSION}"
id="apache-iceberg-go-${VERSION}-rc${RC}"
tar_gz="${id}.tar.gz"
echo "TAR_GZ=${tar_gz}" >> ${GITHUB_ENV}
git archive HEAD --prefix "${id}/" --output "${tar_gz}"
Expand Down Expand Up @@ -91,6 +91,7 @@ jobs:
run: |
tar_gz=$(echo apache-iceberg-go-*.tar.gz)
version=${tar_gz#apache-iceberg-go-}
version=${version%-rc*}
version=${version%.tar.gz}
if [ "${GITHUB_REF_TYPE}" = "tag" ]; then
rc="${GITHUB_REF_NAME#*-rc}"
Expand Down
2 changes: 1 addition & 1 deletion dev/release/verify_rc.sh
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ RC="$2"

ICEBERG_DIST_BASE_URL="https://downloads.apache.org/iceberg"
DOWNLOAD_RC_BASE_URL="https://github.com/apache/iceberg-go/releases/download/v${VERSION}-rc${RC}"
ARCHIVE_BASE_NAME="apache-iceberg-go-${VERSION}"
ARCHIVE_BASE_NAME="apache-iceberg-go-${VERSION}-rc${RC}"

: "${VERIFY_DEFAULT:=1}"
: "${VERIFY_DOWNLOAD:=${VERIFY_DEFAULT}}"
Expand Down

0 comments on commit 8ca0bb7

Please sign in to comment.