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

make build fails #34

Closed
NickLarsenNZ opened this issue Jul 31, 2019 · 3 comments
Closed

make build fails #34

NickLarsenNZ opened this issue Jul 31, 2019 · 3 comments

Comments

@NickLarsenNZ
Copy link
Contributor

NickLarsenNZ commented Jul 31, 2019

When I run the steps as outlined in the README, the build fails on a vendored dependency:

~/go/src/github.com/infobloxopen/terraform-provider-infoblox master$ make build
Doing dep ensure
cannot Stat: stat /home/user/go/src/github.com/infobloxopen/terraform-provider-infoblox/vendor/github.com/hashicorp/go-getter/test-fixtures/detect-file-symlink-pwd/syml/pwd: no such file or directory
GNUmakefile:13: recipe for target 'deps' failed
make: *** [deps] Error 1

I notice that pwd is a symlink to a non existant file in the directory above:

~/go/src/github.com/infobloxopen/terraform-provider-infoblox master$ ls -al vendor/github.com/hashicorp/go-getter/test-fixtures/detect-file-symlink-pwd/syml/pwd
lrwxrwxrwx 1 user user 7 Jul 29 16:22 vendor/github.com/hashicorp/go-getter/test-fixtures/detect-file-symlink-pwd/syml/pwd -> ../real

To get around it, I do the following:

touch vendor/github.com/hashicorp/go-getter/test-fixtures/detect-file-symlink-pwd/real

Finally:

~/go/src/github.com/infobloxopen/terraform-provider-infoblox master$ make build
==> Checking that code complies with gofmt requirements...
go install
go build -o terraform-provider-infoblox

I think by tagging and producing release artifacts using CI/CD as discussed in #33, users will not be tripped up by this. I also think by moving away from vendoring (to go modules), the problem will disappear, the repo will be smaller, and the dependency download process will be much faster.

@saiprasannasastry
Copy link
Contributor

hi , will try to incorporate the comments, this might take some time. right now we have no preferred CI tool, will have to go through both and investigate

@saiprasannasastry
Copy link
Contributor

@NickLarsenNZ like you were saying in previous issue , if you could raise a PR that would help us as well

AvRajath added a commit that referenced this issue Dec 16, 2020
@somashekhar
Copy link
Contributor

Can you once check on the issue ? As of now we see build is successful.

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

No branches or pull requests

3 participants