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

fetched dependencies missing content of their submodules #698

Closed
dt opened this issue Dec 2, 2016 · 8 comments
Closed

fetched dependencies missing content of their submodules #698

dt opened this issue Dec 2, 2016 · 8 comments

Comments

@dt
Copy link

dt commented Dec 2, 2016

Demo repo here:
https://github.com/dt/glide-fetch-submodule

Essentially, github.com/dt/subparentdemo is a little demo library that uses a git submodule for one of its internal libraries.

When glide fetches subparentdemo it doesn't fetch its submodule subdemo so that subpackage, which subparentdemo depends on, is missing.

@sdboyer
Copy link
Member

sdboyer commented Dec 2, 2016

Yeah, I think it's pretty unequivocal that we have to do this. All the git handling needs to be made recursive.

That's more or less easy for clones and fetches, but I'm concerned about how it's going to affect our checkout logic for within the installer.

gps suffers from this same issue...it could benefit from the same fixes in Masterminds/vcs.

@dt
Copy link
Author

dt commented Dec 2, 2016

fwiw, gvt and govendor appear to have the same issue, e.g. FiloSottile/gvt#81

@sdboyer
Copy link
Member

sdboyer commented Dec 4, 2016

well i think those fixes in Masterminds/vcs should do it

@alexanderbez
Copy link

Is there a resolution on this? I'm facing a similar issue with one of my packages rendering the pkg unusable.

$ glide --version
glide version 0.12.3

@dt
Copy link
Author

dt commented Apr 17, 2017

@alexanderbez deps that include submodules seem to be handled correctly, at least in my usage, since #748 (though that isn't in a "release" yet AFAIK).

@alexanderbez
Copy link

Ahhh thanks @dt. Seems you're right -- looks like the last release was on Oct 3, 2016.I take it'll be in the next release then?

Just curious, how did you use the latest from master then? Just building locally?

@dt
Copy link
Author

dt commented Apr 17, 2017

@alexanderbez a simple go get -u gitHub.com/Masterminds/glide should do the trick if you want latest master.

If you want to get fancy and eliminate risk of a broken master biting you, I vendor glide at a known-works-for-me version along with everything else though and then just go install ./vendor/github.com/Masterminds/glide along with other dev tools.

@alexanderbez
Copy link

Awesome. Thanks @dt. This ticket should be closed then I suppose.

@sdboyer sdboyer closed this as completed Apr 18, 2017
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