-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Fix/texts #11298
Fix/texts #11298
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to run "make gen" to address the failing CI test
When I run chain/actors/policy/policy.go
itests/kit/ensemble_opts_nv.go
go run ./gen/api
goimports -w api
bash: line 1: goimports: command not found
make: *** [Makefile:308: api-gen] Error 127 So then installed goimports with Then when trying to install goimports with # go get -v golang.org/x/tools/cmd/goimports
go: github.com/filecoin-project/filecoin-ffi@v0.30.4-0.20220519234331-bfd1f5f9fe38 (replaced by ./extern/filecoin-ffi): reading extern/filecoin-ffi/go.mod: open /go/filecoin-project_lotus/extern/filecoin-ffi/go.mod: no such file or directory So finally I built and installed goimports from source which made go run ./gen/api
goimports -w api
goimports -w api
go run ./gen/main.go
chain/consensus/utils.go:12:2: github.com/filecoin-project/filecoin-ffi@v0.30.4-0.20220519234331-bfd1f5f9fe38 (replaced by ./extern/filecoin-ffi): reading extern/filecoin-ffi/go.mod: open /root/filecoin-project_lotus/extern/filecoin-ffi/go.mod: no such file or directory
chain/vm/fvm.go:21:2: github.com/filecoin-project/filecoin-ffi@v0.30.4-0.20220519234331-bfd1f5f9fe38 (replaced by ./extern/filecoin-ffi): reading extern/filecoin-ffi/go.mod: open /root/filecoin-project_lotus/extern/filecoin-ffi/go.mod: no such file or directory
make: *** [Makefile:288: type-gen] Error 1 |
After installing goimports with go install, you need to make sure its install location is in your $PATH. You can run |
I had to make some extra steps for it to work, but finally go it. Adding this note here in case it helps anybody else in the future, but it maybe it should be added to the Contribute page. First, an issue was that this repo has submodules so a standard Second I had to build the module filecoin-ffi first: cd extern/filecoin-ffi
make Note: that requires to install Then a couple of dependencies are required: apt install libhwloc-dev
apt install ocl-icd-opencl-dev Finally after all this, |
After running |
Related Issues
No issue opened
Proposed Changes
Multiple typos on files