-
-
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
remove Godeps #5440
remove Godeps #5440
Conversation
5bd1033
to
0c43c2c
Compare
65eeda0
to
8aeaaa5
Compare
fixes #2722 License: MIT Signed-off-by: Steven Allen <[email protected]>
8aeaaa5
to
0252710
Compare
test/3nodetest/Makefile
Outdated
@@ -20,7 +19,8 @@ data/filerand: bin/random | |||
./bin/random 50000000 > ./data/filerand | |||
|
|||
bin/random: $(RANDOMSRC)/**/*.go | |||
go build -o ./bin/random $(RANDOMSRC)/random | |||
go get github.com/jbenet/go-random | |||
go build -o ./bin/random github.com/jbenet/go-random/random |
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.
This is the only real change. It uses go-get, not gx, but, as far as I can tell, we don't use these tests at the moment anyways.
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.
Isn't this also https://github.com/ipfs/go-ipfs/pull/5440/files#diff-c8660427d0699f8fe97cb9943c5bd91eR40? I'd just depend on that
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.
Other than the go-random thing, looks good
test/3nodetest/Makefile
Outdated
@@ -20,7 +19,8 @@ data/filerand: bin/random | |||
./bin/random 50000000 > ./data/filerand | |||
|
|||
bin/random: $(RANDOMSRC)/**/*.go | |||
go build -o ./bin/random $(RANDOMSRC)/random | |||
go get github.com/jbenet/go-random | |||
go build -o ./bin/random github.com/jbenet/go-random/random |
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.
Isn't this also https://github.com/ipfs/go-ipfs/pull/5440/files#diff-c8660427d0699f8fe97cb9943c5bd91eR40? I'd just depend on that
License: MIT Signed-off-by: Steven Allen <[email protected]>
SGTM. I was stuck trying to figure out the right gx hash (the makefile is independent of our make system) but I realized I can just use the other make system. LGTY? |
fixes #2722