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

Build failure (gx dependencies) #29

Closed
mildred opened this issue Mar 24, 2016 · 1 comment
Closed

Build failure (gx dependencies) #29

mildred opened this issue Mar 24, 2016 · 1 comment
Labels
help wanted Seeking public contribution on this issue

Comments

@mildred
Copy link

mildred commented Mar 24, 2016

Since commit 7b48e5b I am unable to build this package. More specifically ./p2p/net/swarm/addr. The first problem is:

# github.com/ipfs/go-libp2p/p2p/net/swarm/addr
p2p/net/swarm/addr/addr.go:70: cannot use split[0] (type "gx/ipfs/QmcobAGsCjYt5DXoq9et9L8yR8er7o7Cu3DTvpaq12jYSz/go-multiaddr".Multiaddr) as type "github.com/jbenet/go-multiaddr".Multiaddr in argument to manet.IsIP6LinkLocal:
    "gx/ipfs/QmcobAGsCjYt5DXoq9et9L8yR8er7o7Cu3DTvpaq12jYSz/go-multiaddr".Multiaddr does not implement "github.com/jbenet/go-multiaddr".Multiaddr (wrong type for Decapsulate method)
        have Decapsulate("gx/ipfs/QmcobAGsCjYt5DXoq9et9L8yR8er7o7Cu3DTvpaq12jYSz/go-multiaddr".Multiaddr) "gx/ipfs/QmcobAGsCjYt5DXoq9et9L8yR8er7o7Cu3DTvpaq12jYSz/go-multiaddr".Multiaddr
        want Decapsulate("github.com/jbenet/go-multiaddr".Multiaddr) "github.com/jbenet/go-multiaddr".Multiaddr

When I look at the code, the failure is normal. The package gx/ipfs/QmQB7mNP3QE7b4zP2MQmsyJDqG5hzYE2CL8k1VyLWky2Ed/go-multiaddr-net uses github.com/jbenet/go-multiaddr while libp2p uses the incompatible package gx/ipfs/QmcobAGsCjYt5DXoq9et9L8yR8er7o7Cu3DTvpaq12jYSz/go-multiaddr.

Both are very similar but the Multiaddr type from the gx package is different from the Multiaddr type from the github package.

We should probably update gx/ipfs/QmQB7mNP3QE7b4zP2MQmsyJDqG5hzYE2CL8k1VyLWky2Ed/go-multiaddr-net to use gx/ipfs/QmcobAGsCjYt5DXoq9et9L8yR8er7o7Cu3DTvpaq12jYSz/go-multiaddr instead of github.com/jbenet/go-multiaddr

Also, this might not be the right place for this discussion, but I am worrying about the go package names that encode the download location. Both github and gx. Go has a feature to enable any domain (not just GitHub) to be part of an import path (see Remote import paths). Perhaps we should use the ipfs.io domain for our packages and have gx integrate into that scheme. The package name would be the same, regardless of where it comes from, IPFS or GitHub.

@whyrusleeping whyrusleeping added the help wanted Seeking public contribution on this issue label Sep 14, 2016
@whyrusleeping
Copy link
Contributor

I think this has since been resolved.

@whyrusleeping whyrusleeping added help wanted Seeking public contribution on this issue and removed help wanted Seeking public contribution on this issue libp2p-backlog labels Sep 14, 2016
marten-seemann added a commit that referenced this issue Apr 22, 2022
generate certificates that are valid for 180 days
marten-seemann added a commit that referenced this issue Apr 22, 2022
change OpenStream to accept a context
marten-seemann pushed a commit that referenced this issue Jul 1, 2022
Correctly handle stream errors
marten-seemann pushed a commit that referenced this issue Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Seeking public contribution on this issue
Projects
None yet
Development

No branches or pull requests

3 participants