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

Getting dependencies with "..." adds unnecessary packages #24

Open
pkieltyka opened this issue Aug 18, 2014 · 5 comments
Open

Getting dependencies with "..." adds unnecessary packages #24

pkieltyka opened this issue Aug 18, 2014 · 5 comments

Comments

@pkieltyka
Copy link

Using ... at the end of a go get, will fetch all of the packages under that path. But in some projects this will import unnecessary packages.. for example like, https://github.com/garyburd/go-oauth. Adding that to your Goopfile will try to fetch the "examples" package and add the appengine as a dependency, which clear is not a good thing :|

@pkieltyka
Copy link
Author

fixed with: pkieltyka@c60be07

.. this does change Goop a bit, since it won't just magically grab every dependency under the sun.. but thats a bad thing to do anyways in my opinion..

@pkieltyka
Copy link
Author

I'd love to get another opinion on this, but I think its best to be explicit about go geting

is instead for the developer to specify the "..." at the end of a repo definition in the Goopfile themselves instead of this being implied for every entry.

ie. in a Goopfile

github.com/garyburd/go-oauth/oauth #sometag

vs.

github.com/garyburd/go-oauth/oauth... #sometag

@zeeyang
Copy link

zeeyang commented Dec 19, 2014

+1 for explicit dependency path. I'm having a problem with circular dependency and came across this patch. This doesn't solve dependency conflicts, but the default "grab everything" behavior mades things much worse.

@pkieltyka
Copy link
Author

Yea goop sucks. Lots of issues and short comings.

On Friday, December 19, 2014 at 2:51 AM, Zee Yang wrote:

+1 for explicit dependency path. I'm having a problem with circular dependency and came across this patch. This doesn't solve dependency conflicts, but the default "grab everything" behavior mades things much worse.


Reply to this email directly or view it on GitHub (#24 (comment)).

@zeeyang
Copy link

zeeyang commented Dec 19, 2014

I don't think the ... notation is necessary. I would prefer to have a goop introspect command that go through import headers and build the Goopfile. It should give you the precise dependencies and stay close to Go's import conventions.

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