This is an automated email from the ASF dual-hosted git repository.
kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new 354a22fa92 MINOR: [Release] Proactively remove java-jni-dist directory
in verification script (#44718)
354a22fa92 is described below
commit 354a22fa92b4a745adcc0f23bd89753f7af17f2f
Author: Bryce Mecum <[email protected]>
AuthorDate: Wed Nov 13 19:14:50 2024 -0800
MINOR: [Release] Proactively remove java-jni-dist directory in verification
script (#44718)
### Rationale for this change
If you set `ARROW_TMPDIR` and run `TEST_DEFAULT=0 TEST_INTEGRATION=1
dev/release/verify-release-candidate.sh` repeatedly, you'll run into an error:
> mv: rename
/Users/bryce/src/apache/arrow/arrow-tempdir//java-jni-install/lib/arrow_cdata_jni
to /Users/bryce/src/apache/arrow/arrow-tempdir//java-jni-dist/arrow_cdata_jni:
Directory not empty
### What changes are included in this PR?
In the `test_package_java` step in the verification script, I added a `rm
-fr `to get rid of java-jni-dist dir so the following `mv` command works.
### Are these changes tested?
Yes, locally.
### Are there any user-facing changes?
No.
Authored-by: Bryce Mecum <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
---
dev/release/verify-release-candidate.sh | 1 +
1 file changed, 1 insertion(+)
diff --git a/dev/release/verify-release-candidate.sh
b/dev/release/verify-release-candidate.sh
index 17d10601d1..0305e22843 100755
--- a/dev/release/verify-release-candidate.sh
+++ b/dev/release/verify-release-candidate.sh
@@ -577,6 +577,7 @@ test_package_java() {
normalized_arch=x86_64
;;
esac
+ rm -fr ${dist_dir}
mkdir -p ${dist_dir}
mv ${install_dir}/lib/* ${dist_dir}
mvn install \