-
Notifications
You must be signed in to change notification settings - Fork 266
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
Place dependencies into separate file #2838
Conversation
Probably quite a lot of this could also be moved into separate |
Thanks @K20shores and @ZedThree. Hope everybody stayed/is staying warm. Now that things are thawing out in rural Oregon, I should be able to resume review of the PR backlog. |
One last change occurs to me; the new file needs to be added to the |
I can make that change; other than that, this looks great, thanks! |
@WardF |
If it includes the directory we should be good. I'll double check to be sure, but that's probably got it. Thanks! |
This separate outs the dependencies and partially addresses #2713
find_package
calls are now in thedependencies.cmake
file, making it easyish to see in one place what is needed