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

Only load required software definitions #203

Closed
sethvargo opened this issue Jun 23, 2014 · 3 comments · Fixed by #264
Closed

Only load required software definitions #203

sethvargo opened this issue Jun 23, 2014 · 3 comments · Fixed by #264
Labels
Status: Good First Issue An issue ready for a new contributor. Type: Enhancement Adds new functionality.

Comments

@sethvargo
Copy link
Contributor

We often have features in software dependencies that cause builds to fail, but those softwares aren't actually part of the build. For example, if you create a software definition like:

raise Exception

and commit to omnibus-software on master, all builds will fail. Omnibus should only load the required software definitions and leave the other ones alone. This will reduce the number of deprecation warnings and errors like #202.

@lamont-granquist
Copy link
Contributor

+1

@lamont-granquist
Copy link
Contributor

also only load the project definition for the project that you're actually building.

@sethvargo
Copy link
Contributor Author

Started some work on #264

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Good First Issue An issue ready for a new contributor. Type: Enhancement Adds new functionality.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants