-
Notifications
You must be signed in to change notification settings - Fork 34
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
Install is broken #7
Comments
Install works for me using the spells I added to the readme. You can't do "go get"?
…Sent from my iPhone
On Jul 5, 2017, at 12:35 PM, loicgelle ***@***.***> wrote:
As stated in the readme, install is broken. I would suggest that major changes like this take place in a different branch, and are merged once everything is fixed. It made my Docker package builds fail
@ianamason
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I was talking about the first install. Not a big deal to fix but it is quite a change |
Yes well go is very opinionated about how things are structured. Quite painful it was to get it to work. Very poorly documented. I am finished with major changes.
…Sent from my iPhone
On Jul 5, 2017, at 12:48 PM, loicgelle ***@***.***> wrote:
I was talking about the first install. git clone ... does not work anymore, you have to clone into GOPATH/src/github.com/SRI-CSL.
Not a big deal to fix but it is quite a change
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@ianamason I still leave the issue open as a remainder to fix the README definitely |
Yes please. Why do you need to clone in docker? can't you do the "go get"s and |
This is what I am doing now indeed |
well if you look at the imports in the executable files, you see why go forces us to |
@ianamason I know that, I struggled to understand their weird import system at first |
I will close this when the README.md is polished up. |
Thanks Loic for the environmment fixes. Bruno and I will rewrite the README.md, |
As stated in the readme, install is broken. I would suggest that major changes like this take place in a different branch, and are merged once everything is fixed. It made my Docker package builds fail
@ianamason
The text was updated successfully, but these errors were encountered: