-
Notifications
You must be signed in to change notification settings - Fork 25
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
Please include installation instructions #10
Comments
I added a section to the readme on installation, please have a look and let me know if you have any questions. |
❯❯❯ GOPATH=~/.go go get -u github.com/brentp/goleft/...
package github.com/brentp/goleft/emdepth/mops: found packages mops (mops.go) and emdepth (mops_test.go) in /Users/sjackman/.go/src/github.com/brentp/goleft/emdepth/mops
❯❯❯ which goleft
goleft not found
❯❯❯ ls ~/.go/bin
ls: /Users/sjackman/.go/bin: No such file or directory |
|
I can put the main into cmd/goleft/main.go and then |
Please see the updated readme and re-open this issue if you have any problems. Thanks for bringing this to my attention. |
I'd suggest add export GOPATH=~/.go
PATH="$GOPATH/bin:$PATH"
go get -u github.com/brentp/goleft/...
go install github.com/brentp/goleft/cmd/goleft
goleft --help
Is that warning expected?
How do I build Thanks for your help, Brent. |
I'm hesitant to add more than I already have since most users would and should just copy the provided binary to their yes: yes: I'll re-open this as a placeholder with the intent of adding an |
Perhaps I'm being pedantic, but
The
If so, then the installation instructions should include that step. It's currently not clear where the executable is installed, and it's not easy to copy an executable if you don't know where it is. A suggestion:
Finally, copy the |
I see what you mean, I have made another commit to clarify the point about install vs from source. Installing I'm more sticky on all of this because I want to promote that the preferred way to use is to download the binary. That said, I want it to be used so I'll accept PR's that clarify anything I've overlooked. |
Thanks! A formula for Homebrew-science would be great.
The text was updated successfully, but these errors were encountered: