Skip to content
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 issue #5733

Closed
fedor-shi opened this issue Nov 4, 2018 · 4 comments · Fixed by #5747
Closed

Install issue #5733

fedor-shi opened this issue Nov 4, 2018 · 4 comments · Fixed by #5747
Assignees

Comments

@fedor-shi
Copy link

fedor-shi commented Nov 4, 2018

Version information:

Software:

    System Software Overview:

      System Version: macOS 10.13.6 (17G65)
      Kernel Version: Darwin 17.7.0
      Boot Volume: Macintosh HD
      Boot Mode: Normal
$ go version
go version go1.11.1 darwin/amd64

Type:

Bug

Description:

Get the error, when try to install

make install
bin/check_go_version 1.11
fatal: 'go version' failed with output: go: parsing $GOFLAGS: non-flag "\"github.com/ipfs/go-ipfs\".CurrentCommit=3f7668bdc\""
make: *** [check_go_version] Error 1

executed from

 $pwd
/Users/theodor/go/src/github.com/ipfs/go-ipfs

 ~/go/src/github.com/ipfs/go-ipfs   master                                                                                                                                                                            ✔  1294  11:51:59
 $ echo $GOPATH
/Users/theodor/go
@eingenito
Copy link
Contributor

I don't know a ton about the build process, but it seems like you're picking up some crud in your GOFLAGS environment variable that's blowing up the call to bin/check_go_version. I don't know how our build uses (or might populate) GOFLAGS, but if you just do echo $GOFLAGS is there anything in there? If you run bin/check_go_version 1.11 in the go-ipfs directory what does it explode? I'll take a look at the makefiles and see if I can figure out how it might have set that variables in a bad way.

@fedor-shi
Copy link
Author

@eingenito you are right
I have

echo $GOFLAGS
-ldflags=-compressdwarf=false

and it breaks the build. It works for me in a such way make install GOFLAGS=

@Stebalien Stebalien reopened this Nov 6, 2018
Stebalien added a commit that referenced this issue Nov 6, 2018
We want to explicitly pass these where we need them.

fixes #5733

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
@ghost ghost assigned Stebalien Nov 6, 2018
@ghost ghost added the status/in-progress In progress label Nov 6, 2018
@Stebalien
Copy link
Member

Does #5747 also fix your issue? We shouldn't be failing in this case regardless.

@fedor-shi
Copy link
Author

@Stebalien I tested it, works with fix

@ghost ghost removed the status/in-progress In progress label Nov 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants