-
Notifications
You must be signed in to change notification settings - Fork 166
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
Enable CMake integration with FetchContent #260
base: main
Are you sure you want to change the base?
Conversation
Hi @jeanchristopheruel , thanks for proposing and implementing these changes. This is how I think we should do it. All CMake changes related to
I see that you have changed options from Also, I will not rely on this fetching mechanism of CMake for installing dependencies. I rely on conda (conda-forge more exactly) as a dependency manager, which you should give it a try. A common misconception is that conda is for python only; I use conda to install even compilers! Have a look at the environment.devenv.yml file. |
Interesting indeed. I'm not that familiar with conda. Personally, I think FetchContent makes the code more portable and avoids adding dependencies. Thank you for your answer, I will suggest an improvement. |
Integrate autodiff in CMake using FetchContent
And for more granular control:
Fixes #259