Skip to content
This repository was archived by the owner on Sep 9, 2020. It is now read-only.

status: collectConstraints() should collect constraints from root project too #1452

Closed
darkowlzz opened this issue Dec 13, 2017 · 5 comments · Fixed by #1869
Closed

status: collectConstraints() should collect constraints from root project too #1452

darkowlzz opened this issue Dec 13, 2017 · 5 comments · Fixed by #1869

Comments

@darkowlzz
Copy link
Collaborator

collectConstraints() in status.go collects constraints declared by all the dependencies. This results in a dependency graph. But since we are not considering the constraints set in the root project, this results in an incomplete graph.

Constraints from manifest should be read and included in the returned constraintsCollection.

This can be done with #1409 or separately.

@JamieCrisman
Copy link
Contributor

I'm interested in this one, but I'll probably have some questions for it in the slack.

@darkowlzz
Copy link
Collaborator Author

@JamieCrisman Great!
But note that there's a PR that's about to change the code around this a little #1413. So, look out for that :)

@JamieCrisman
Copy link
Contributor

Will be looking at this more now that #1413 was merged.

@JamieCrisman
Copy link
Contributor

Going to need to release for someone else to grab. Sorry for holding it up.

@sdboyer
Copy link
Member

sdboyer commented Dec 26, 2017

no problem - thanks for communicating about it!

tinnefeld added a commit to tinnefeld/dep that referenced this issue May 20, 2018
The collectConstraints() function in status.go incorporates effective
constraints from the root projects manifest. Issue golang#1452.
tinnefeld added a commit to tinnefeld/dep that referenced this issue Jun 4, 2018
The collectConstraints() function in status.go incorporates
constraints from the root projects manifest. Issue golang#1452.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants