-
Notifications
You must be signed in to change notification settings - Fork 79
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
dh-make-golang no longer able to calculate dependencies of package with vendoring (as of golang-1.6.2?) #46
Comments
I think the relevant change might be:
|
Thank you @stapelberg! Yes, setting I took a look at the dh-make-golang, and Ithink I will make a pull request to |
How does this work now in Go 1.7? It seems my I'm new to building packages in general, so would appreciate some help. |
@Baisang Can you file a separate issue please with the command you’re running, the version of dh-make-golang you’re using and the output you’re seeing? In general, the program works. |
Had a chance to reproduce this issue using @Baisang’s example invocation The main problem is Line 246 in be72733
golang.org/x/tools/go/vcs to golang.org/x/tools , which we then convert into a Debian package name.
When using vendoring, the VCS information needs to be determined differently. In the particular case of I’m not sure how other vendoring tools deal with this and whether there’s a spec — any pointers welcome. I’ll file an issue with github.com/golang/dep to make sure our requirements are known to the hopefully-soon canonical vendoring tool. For the moment, one either needs to locate all dependencies manually, or un-vendor the package (possibly in a private fork on GitHub) before packaging. |
Hey, I've been bitten by this because I need to package https://github.com/mheese/journalbeat for work, and that has vendored all its dependencies. What I've done is just nuking the vendor directory, and reinstalling the dependencies with https://github.com/olasd/dh-make-golang/tree/bugfix/vendoring has my work, feel free to pull from it if the approach makes sense. |
@olasd Thanks, I like that solution. It should fix the issue independent of the vendoring tool used. Pulled (and cleaned up a little). |
One last thing I would like to point out for any future readers - you will have to correct for dependencies that have different names/that are sub-packages, e.g. I had imported What I ended up doing was just requiring |
@Baisang Note that a package including the |
@stapelberg if there is a project that uses a dependency that isn't available in "apt"/doesn't have a corresponding "debian name" (for lack of a better term, sorry if I am mixing up my terminology here), like the Sorry, I feel like there is a key point I am not understanding here. I haven't been able to find much literature on what I'm trying to do (building a debian package for golang project that uses vendoring), so I really appreciate your help thus far! |
The dependencies need to be packaged first. |
I discovered yesterday that the command:
is no longer able to calculate the necessary dependent Go packages in debian/control. The govender repository contains vendoring in the
vendor/
directory.But I remember that it worked back on 2016-05-01 (a little more than a month ago) when I created the initial package for govendor, and
dh-make-golang
has not changed since. Could it have anything to do with recent changes in golang-go packaging, especially the new go1.6.2?(I have yet to investigate in depth, but filing an issue so I don't forget.)
The text was updated successfully, but these errors were encountered: