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 \

Reply via email to