Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

go mod tidy for lotus-soup testplans #6124

Merged
merged 1 commit into from
Apr 29, 2021
Merged

go mod tidy for lotus-soup testplans #6124

merged 1 commit into from
Apr 29, 2021

Conversation

nonsense
Copy link
Member

Fixup for #6122


Since CircleCI runs only go get lotus@master, but not go mod tidy, sometimes we have to clean the go.mod of old libraries in order to avoid compile errors:

Step 15/22 : RUN cd ${PLAN_DIR}     && go env -w GOPROXY="${GO_PROXY}"     && CGO_ENABLED=${CgoEnabled} GOOS=linux GOARCH=amd64 go build -o ${PLAN_DIR}/testplan.bin ${BUILD_TAGS} ${TESTPLAN_EXEC_PKG}
 ---> Running in d0cd81ce0cd5
�[91m/go/pkg/mod/github.com/filecoin-project/[email protected]/api/api_full.go:14:2: missing go.sum entry for module providing package github.com/filecoin-project/go-data-transfer (imported by github.com/filecoin-project/lotus/api); to add:
	go get github.com/filecoin-project/lotus/[email protected]
/go/pkg/mod/github.com/filecoin-project/[email protected]/node/modules/client.go:13:2: missing go.sum entry for module providing package github.com/filecoin-project/go-data-transfer/channelmonitor (imported by github.com/filecoin-project/lotus/node/modules); to add:
	go get github.com/filecoin-project/lotus/node/[email protected]
/go/pkg/mod/github.com/filecoin-project/[email protected]/node/modules/client.go:14:2: missing go.sum entry for module providing package github.com/filecoin-project/go-data-transfer/impl (imported by github.com/filecoin-project/lotus/node/modules); to add:
	go get github.com/filecoin-project/lotus/node/[email protected]
...

@magik6k magik6k merged commit 492b026 into master Apr 29, 2021
@magik6k magik6k deleted the testplans-go-mod branch April 29, 2021 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants