Skip to content

Commit f6e48b4

Browse files
authored
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]>
1 parent bab5580 commit f6e48b4

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

dev/release/verify-release-candidate.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1027,7 +1027,9 @@ TEST_SUCCESS=no
10271027

10281028
setup_tempdir
10291029
ensure_source_directory
1030-
test_source_distribution
1030+
# Run source tests in a subshell so environment variables
1031+
# set for source testing aren't exposed to the binary tests.
1032+
(test_source_distribution)
10311033
test_binary_distribution
10321034

10331035
TEST_SUCCESS=yes

0 commit comments

Comments
 (0)