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

No vendored dependencies #7

Closed
nathanleclaire opened this issue Nov 11, 2015 · 7 comments
Closed

No vendored dependencies #7

nathanleclaire opened this issue Nov 11, 2015 · 7 comments
Assignees

Comments

@nathanleclaire
Copy link
Contributor

The Makefile and everything uses Godeps but the dependencies are not vendored there.

@zchee
Copy link
Member

zchee commented Nov 12, 2015

Hi @nathanleclaire

Sorry, I actually do not know well the godep...
What does it mean in particular?

@zchee
Copy link
Member

zchee commented Nov 12, 2015

@nathanleclaire Maybe libguestfs/guestfs?

@zchee zchee self-assigned this Nov 12, 2015
@nathanleclaire
Copy link
Contributor Author

Usually the required libraries are bundled alongside the code inside of Godeps. For instance, take a look at the Machine source code. Inside of Godeps/_workspace/src there are all the packages we depend on. I haven't been able to successfully compile your code from the instructions here so far.

@zchee
Copy link
Member

zchee commented Nov 12, 2015

@nathanleclaire Oh, I understand!

I use the godep only to the management of package vendoring for docker-machine-xhyve.
but, I had a misunderstanding that godep go build using the GO15VENDOREXPERIMENT=1.

I think How to solve this problem...

  1. Put GO15VENDOREXPERIMENT=1 to Makefile, like docker-machine
  2. Symlink vendor to Godeps/_workspace/src for not set GO15VENDOREXPERIMENT=1 user or Go 1.4 or previous.
  3. Move vendor to Godeps/_workspace/src, and symlink Godeps/_workspace/src to vendor for GO15VENDOREXPERIMENT=1 user.

Which do you think is correct?

BTW,

Inside of Godeps/_workspace/src there are all the packages we depend on.

Do Godeps/_workspace/src exists in the docker-machine?
https://github.com/docker/machine/tree/master/Godeps

@zchee
Copy link
Member

zchee commented Nov 12, 2015

@nathanleclaire
The Godeps/_workspace/src does not exist, I was referring to the directory structure of the docker-machine

@nathanleclaire
Copy link
Contributor Author

Ah, OK, you have that vendor directory -- didn't see that. In that case it might be sufficient to set GO15VENDOREXPERIMENT=1 in Makefile.

@nathanleclaire
Copy link
Contributor Author

With that setting I can compile the binary properly but running into a variety of other issues.

zchee added a commit that referenced this issue Nov 12, 2015
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

2 participants