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

update filecoin-ffi in go mod #4584

Merged
merged 1 commit into from
Oct 26, 2020
Merged

Conversation

jsign
Copy link
Contributor

@jsign jsign commented Oct 24, 2020

Currently, the go.mod has an old version of this dependency.
For this repo isn't a problem since it has a replace directive, but for any client go mod tidy fails.

Minimal way to reproduce the problem now:

package main

import (
        "context"
        "github.com/filecoin-project/lotus/api/apistruct"
)

func main() {
        var api apistruct.FullNodeStruct
        _ = api
  }

Run go mod tidy:

go: finding module for package github.com/filecoin-project/specs-actors/actors/abi
github.com/jsign/lotusclientest imports
        github.com/filecoin-project/lotus/api/apistruct imports
        github.com/filecoin-project/go-fil-markets/storagemarket tested by
        github.com/filecoin-project/go-fil-markets/storagemarket.test imports
        github.com/filecoin-project/go-fil-markets/pieceio imports
        github.com/filecoin-project/filecoin-ffi imports
        github.com/filecoin-project/specs-actors/actors/abi: module github.com/filecoin-project/specs-actors@latest found (v0.9.13), but does not contain package github.com/filecoin-project/specs-actors/actors/abi

Generated go.mod:

module github.com/jsign/lotusclientest

go 1.15

require (
        github.com/filecoin-project/lotus v1.1.2
)

In Poweragte I work around this with a replace, but other users might not have a clue of how to fix it. Also, if anyone else is using Powergate as a lib they should also replace in their go.mod since replaces doesn't bubble up modules, which is a bigger problem.

This PR fixes this problem.

Signed-off-by: Ignacio Hagopian <[email protected]>
@jsign jsign marked this pull request as ready for review October 24, 2020 20:30
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