Skip to content

windows#943

Closed
cryptix wants to merge 1 commit intomasterfrom
windows
Closed

windows#943
cryptix wants to merge 1 commit intomasterfrom
windows

Conversation

@cryptix
Copy link
Contributor

@cryptix cryptix commented Mar 19, 2015

This updates go-reuseport to get windows compilation.

To compile on windows, you need to disable fuse by compiling with go build -tags nofuse.

There are a number of other outstanding issues:

  • when build for windows, github.com/cheggaaa/pb depends on github.com/olekukonko/ts which is currently not vendored
  • shutdown sometimes leaves daemon.lock sitting around and spits out ERROR ipnsfs <autogenerated>:24: leveldb: closed
  • sometimes blockservice.test.exe or dht.test.exe hang during go test -tags nofuse ./... and need to be killed manually
  • some test failures- runs: 1 2
  • some weird inconsistency between running in cmd.exe and bash.exe from git for windows

And I'm sure there is more. My tests are from a windows 7 QemuVM with a flacky VDE/SlirpDHCP network. I welcome every native windows user willing to test this with open arms.

@btc btc added the status/in-progress In progress label Mar 19, 2015
@jbenet
Copy link
Member

jbenet commented Mar 19, 2015

👏 👏 👏 👏 👏 👏 👏

@cryptix cryptix added the help wanted Seeking public contribution on this issue label Mar 19, 2015
This was referenced Mar 19, 2015
@cryptix
Copy link
Contributor Author

cryptix commented Mar 23, 2015

Most of these are too broad for one PR, I think. Godeps update in #958. Moved PR notes to Issue #959.

@cryptix cryptix closed this Mar 23, 2015
@btc btc removed the status/in-progress In progress label Mar 23, 2015
@cryptix cryptix deleted the windows branch March 23, 2015 09:04
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

Successfully merging this pull request may close these issues.

3 participants