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

Use Go's new standard vendoring in 1.6+ #728

Closed
pkieltyka opened this issue Apr 7, 2016 · 13 comments
Closed

Use Go's new standard vendoring in 1.6+ #728

pkieltyka opened this issue Apr 7, 2016 · 13 comments
Milestone

Comments

@pkieltyka
Copy link

I also suggest https://github.com/kardianos/govendor over Godeps.

@pkieltyka pkieltyka changed the title FYI, for go 1.6+ vendor your deps Use Go's new standard vendoring in 1.6+ Apr 7, 2016
@cyphar
Copy link
Member

cyphar commented Apr 7, 2016

Sure, that's fine. But I think we should do a subtree merge or something like that to make sure that the history of vendoring isn't lost.

@crosbymichael crosbymichael modified the milestone: 1.0.0 May 25, 2016
@crosbymichael crosbymichael modified the milestones: 1.1.0, 1.0.0 Jun 1, 2016
@cyphar
Copy link
Member

cyphar commented Jun 8, 2016

I just tried govendor and it doesn't appear to work properly (it doesn't look like it correctly deals with nested requirements of packages and it also doesn't clean up unused files). Currently Docker's hack/vendor.sh does a better job. I'm going to take a look at govend and some others.

@LK4D4
Copy link
Contributor

LK4D4 commented Jun 8, 2016

@cyphar I'm pretty sure that godep and glide are best vendoring tools now(glide even can vendor docker dependencies). I hate godep workflow, but it cleans unneeded files and that's only reason I prefer it over glide.

@cyphar
Copy link
Member

cyphar commented Jun 8, 2016

@monstermunchkin said that govend might actually work nicely here.

The reason I don't like godep is because updating dependencies to a specific version (like in hack/vendor.sh) is a huge pain. I'd prefer we take the magical shell script from Docker rather than use Godep.

@LK4D4
Copy link
Contributor

LK4D4 commented Jun 8, 2016

Yeah, I tried govend. It's really nice.

@crosbymichael
Copy link
Member

We are now using go 1.6 if anyone would like to do the work of porting over our vendor dependencies to the new schema.

@jessfraz
Copy link
Contributor

jessfraz commented Jun 11, 2016

DIBS
I'm on it right now

@marcosnils
Copy link
Contributor

I think I saw a PR from @cyphar that implemented this?

sent from mobile
On Jun 10, 2016 9:04 PM, "Jess Frazelle" notifications@github.com wrote:

DIBS

On Fri, Jun 10, 2016 at 5:00 PM -0700, "Michael Crosby" <
notifications@github.com> wrote:

We are now using go 1.6 if anyone would like to do the work of porting
over our vendor dependencies to the new schema.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#728 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABgV2q020XXCIbEm8EKHRg2FyGgJHB0Mks5qKftzgaJpZM4ICFFO
.

@jessfraz
Copy link
Contributor

@marcosnils
Copy link
Contributor

It's from @LK4D4 actually
#898
sent from mobile
On Jun 10, 2016 9:05 PM, "Marcos Lilljedahl" marcosnils@gmail.com wrote:

I think I saw a PR from @cyphar that implemented this?

sent from mobile
On Jun 10, 2016 9:04 PM, "Jess Frazelle" notifications@github.com wrote:

DIBS

On Fri, Jun 10, 2016 at 5:00 PM -0700, "Michael Crosby" <
notifications@github.com> wrote:

We are now using go 1.6 if anyone would like to do the work of porting
over our vendor dependencies to the new schema.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#728 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABgV2q020XXCIbEm8EKHRg2FyGgJHB0Mks5qKftzgaJpZM4ICFFO
.

@crosbymichael
Copy link
Member

@jfrazelle you trying to crash my browser giving me a link with /files on a PR with 18k line changes?

@jessfraz
Copy link
Contributor

jessfraz commented Jun 11, 2016

so I was stuck there and yeah...

@crosbymichael
Copy link
Member

I think we resolved our vendoring issues using vndr that uses the go vendor setup

stefanberger pushed a commit to stefanberger/runc that referenced this issue Sep 8, 2017
Carry: opencontainers#728

Signed-off-by: Rob Dolin <robdolin@microsoft.com>
Signed-off-by: Qiang Huang <h.huangqiang@huawei.com>
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

6 participants