Skip to content

Commit 1d4b757

Browse files
authored
GH-512: [Release] Fix svn add target (#513)
Fixes GH-512. We can't use `svn add .` here. If we use `.`, we'll get the following error: svn: warning: W150002: '/tmp/arrow-java/dev/release/dist' is already under version control svn: E200009: Could not add all targets because some targets are already versioned svn: E200009: Illegal target for the requested operation
1 parent 3375a5c commit 1d4b757

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

dev/release/release.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ git push origin "${tag}"
5151
release_id="apache-arrow-java-${version}"
5252
dist_url="https://dist.apache.org/repos/dist/release/arrow"
5353
dist_base_dir="dev/release/dist"
54-
dist_dir="dev/release/dist/${release_id}"
54+
dist_dir="${dist_base_dir}/${release_id}"
5555
echo "Checking out ${dist_url}"
5656
rm -rf "${dist_base_dir}"
5757
svn co --depth=empty "${dist_url}" "${dist_base_dir}"
@@ -62,7 +62,7 @@ gh release download "${rc_tag}" \
6262

6363
echo "Uploading to release/"
6464
pushd "${dist_base_dir}"
65-
svn add .
65+
svn add "${release_id}"
6666
svn ci -m "Apache Arrow Java ${version}"
6767
popd
6868
rm -rf "${dist_base_dir}"

0 commit comments

Comments
 (0)