Skip to content

Commit

Permalink
GH-38200: [CI][Release][Go] Ensure removing all module caches (#38222)
Browse files Browse the repository at this point in the history
### Rationale for this change

Module caches don't have write permission by owner. So we can remove them by `rm -rf`.

### What changes are included in this PR?

Run `go clean -modcache` after all builds.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* Closes: #38200

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
  • Loading branch information
kou committed Oct 12, 2023
1 parent 7047e63 commit bb5ebbf
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions dev/release/verify-release-candidate.sh
Original file line number Diff line number Diff line change
Expand Up @@ -880,8 +880,6 @@ test_go() {
go test ./...
fi
go install -buildvcs=false ./...
go clean -modcache

if [ ${TEST_INTEGRATION_GO} -gt 0 ]; then
pushd arrow/internal/cdata_integration
case "$(uname)" in
Expand All @@ -898,7 +896,7 @@ test_go() {
go build -buildvcs=false -tags cdata_integration,assert -buildmode=c-shared -o ${go_lib} .
popd
fi

go clean -modcache
popd
}

Expand Down Expand Up @@ -1204,7 +1202,7 @@ BUILD_JS=$((${TEST_JS} + ${TEST_INTEGRATION_JS}))
BUILD_GO=$((${TEST_GO} + ${TEST_INTEGRATION_GO}))
TEST_INTEGRATION=$((${TEST_INTEGRATION} + ${TEST_INTEGRATION_CPP} + ${TEST_INTEGRATION_JAVA} + ${TEST_INTEGRATION_JS} + ${TEST_INTEGRATION_GO}))

# Execute tests in a conda enviroment
# Execute tests in a conda environment
: ${USE_CONDA:=0}

# Build options for the C++ library
Expand Down

0 comments on commit bb5ebbf

Please sign in to comment.