This is an automated email from the ASF dual-hosted git repository.

areeve 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 f6e48b48cd GH-47830: [Release] Run RC verification source testing step 
in a subshell (#47831)
f6e48b48cd is described below

commit f6e48b48cd91a9c5e3242c0ba7b6c93c54658e3d
Author: Adam Reeve <[email protected]>
AuthorDate: Mon Oct 20 09:13:57 2025 +1300

    GH-47830: [Release] Run RC verification source testing step in a subshell 
(#47831)
    
    ### Rationale for this change
    
    Isolates environment variable changes made when verifying the source from 
the tests that verify the binaries, to prevent these causing errors like in 
#47830.
    
    ### What changes are included in this PR?
    
    Changes `verify-release-candidate.sh` to run `test_source_distribution` in 
a subshell.
    
    ### Are these changes tested?
    
    I've tested this manually, I'm not sure if there are any existing CI tests 
that run this script on PRs.
    
    ### Are there any user-facing changes?
    
    No
    * GitHub Issue: #47830
    
    Authored-by: Adam Reeve <[email protected]>
    Signed-off-by: Adam Reeve <[email protected]>
---
 dev/release/verify-release-candidate.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/dev/release/verify-release-candidate.sh 
b/dev/release/verify-release-candidate.sh
index 56c671910f..9213cdea01 100755
--- a/dev/release/verify-release-candidate.sh
+++ b/dev/release/verify-release-candidate.sh
@@ -1027,7 +1027,9 @@ TEST_SUCCESS=no
 
 setup_tempdir
 ensure_source_directory
-test_source_distribution
+# Run source tests in a subshell so environment variables
+# set for source testing aren't exposed to the binary tests.
+(test_source_distribution)
 test_binary_distribution
 
 TEST_SUCCESS=yes

Reply via email to