Skip to content

Commit

Permalink
[KYUUBI #6061] Delete svn staging dir before checking out
Browse files Browse the repository at this point in the history
# 🔍 Description
## Issue References 🔗

The release process will fail with the following errors if `SVN_STAGING_DIR`(`${KYUUBI_DIR}/work/svn-dev"`) exist.
```
+ upload_svn_staging
+ svn checkout --depth=empty https://dist.apache.org/repos/dist/dev/kyuubi /home/ubuntu/apache-kyuubi/work/svn-dev
svn: warning: cannot set LC_CTYPE locale
svn: warning: environment variable LC_CTYPE is UTF-8
svn: warning: please check that your locale name is correct
svn: E155016: The working copy database at '/home/ubuntu/apache-kyuubi/work/svn-dev' is missing.
```

## Describe Your Solution 🔧

Delete `SVN_STAGING_DIR` before checking out

## Types of changes 🔖

- [x] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

## Test Plan 🧪

Manually test.

---

# Checklist 📝

- [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes #6061 from pan3793/release-rm.

Closes #6061

3a9d51e [Cheng Pan] Delete svn staging dir before checking out

Authored-by: Cheng Pan <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
  • Loading branch information
pan3793 committed Feb 20, 2024
1 parent 06cc3db commit 9d2ea7c
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions build/release/release.sh
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,7 @@ package() {
}

upload_svn_staging() {
rm -rf "${SVN_STAGING_DIR}"
svn checkout --depth=empty "${SVN_STAGING_REPO}" "${SVN_STAGING_DIR}"
mkdir -p "${SVN_STAGING_DIR}/${RELEASE_TAG}"
rm -f "${SVN_STAGING_DIR}/${RELEASE_TAG}/*"
Expand Down

0 comments on commit 9d2ea7c

Please sign in to comment.