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

How can I add a new package in an existing dependency? #6198

Closed
0xmichalis opened this issue Jan 9, 2018 · 4 comments
Closed

How can I add a new package in an existing dependency? #6198

0xmichalis opened this issue Jan 9, 2018 · 4 comments

Comments

@0xmichalis
Copy link
Contributor

I am trying to add k8s.io/apimachinery/pkg/apis/meta/v1 in order to make test-infraable to use generated informers but I am stumbling across various issues when trying to add the new package with dep.

Initially, dep complains that I cannot add the package because it already exists.

dep ensure -add k8s.io/apimachinery
nothing to -add, k8s.io/apimachinery is already in Gopkg.toml and the project's direct imports or required list

At this point, it's unclear what's the right step. Remove the dependency from Gopkg.toml, retry dep ensure, and dep ensure -add k8s.io/apimachinery? What about Gopkg.lock? Also what's the case with all those test files imported by dep? I know that dep includes them, godep wasn't. Did we remove them manually?

@cblecker @BenTheElder @ixdy

@0xmichalis
Copy link
Contributor Author

Also what's the case with all those test files imported by dep? I know that dep includes them, godep wasn't. Did we remove them manually?

Now I see that there are removed during hack/update-bazel.sh.

@stevekuznetsov
Copy link
Contributor

/cc @sttts @deads2k

One of y'all has probably done this already in a different repo?

@BenTheElder
Copy link
Member

see: #5488 for the tests, we're waiting on dep to fix this, but in the meantime hack/update-bazel.sh handles this since it was already in the workflow after updating deps.

@deads2k
Copy link
Contributor

deads2k commented Jan 10, 2018

/cc @sttts @deads2k

One of y'all has probably done this already in a different repo?

We use glide downstream. @sttts has been experimenting with glide, but as I recall there used to be some pretty severe issues using it to vendor k8s libraries.

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

4 participants