-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 get windows: undefined: listenStream #894
Comments
Hrm... Thanks for this! looks like we have some interesting things going on. First, its looking into a |
yeah, this is the main remaining problem to get it to build in windows. btw, even after we build it on windows, i can almost guarantee we do non-windows friendly things with file paths, etc. getting to windows support is likely more than just building. @SimonDanisch in the meantime, may be worth trying out ipfs in a vm. I should mention there's a boot-to-ipfs vm floating around (thanks @zignig!). I use this makefile to make it: https://gist.github.com/jbenet/76afa70955910a2ed097 which uses a mounted ipfs to fetch it lazily. You'll want to just download the vm first from the gateway (since you dont have ipfs :] ), it should be downloadable from: http://gateway.ipfs.io/ipfs/QmdTmSQtCz9D6vt2AbvQkYo3edFcTYoU7FDmDD7bPWDccA/ipfs.iso (~26MB) and then use virtualbox to boot the iso (see the makefile) |
Thanks! Well I got ipfs to run on ubuntu, so I guess the VM path isn't really worth it ;) |
@SimonDanisch we should make an issue (maybe one already exists) about it, outline the things in a checklist and then take care of them. first up is fixing https://github.com/jbenet/go-reuseport i believe |
I took a look at go-reuseport on windows.. impl_unix.go is like 360 lines long, impl_windows.go is only 20 lines.. So impl_windows has yet to be implemented? Most of the errors stem from the fact that impl_windows.go is missing a bunch of functions. But if you put the functions in there then I get compile errors about function re-declaration. Does the code need some refactoring here? |
@SimonDanisch if you feel experimental, we now have a |
Closing this in favor of #959. |
Hi there,
just wanted to try out go-ipfs and run into this:
The text was updated successfully, but these errors were encountered: