-
Notifications
You must be signed in to change notification settings - Fork 35
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
CMake configuration overhaul #133
Comments
Arguments for going to cmake > 3.15: https://cmake.org/cmake/help/latest/policy/CMP0094.html#policy:CMP0094 |
#132 (comment) for two more points:
|
Some more considerations for the choice of the minimum required cmake version: |
This remark fits to this issue, but feel free to open a separate one: I noticed that the two files LCIOConfig.cmake and LCIOConfigVersion.cmake end up after 'make install' in <CMAKE_INSTALL_PREFIX> directly, while for most (maybe all?) cmake built projects I know they are installed/written to <CMAKE_INSTALL_PREFIX>/lib/cmake |
Yes, definitely part of this issue. I need to find some time to finalize #140 and then this should also be fixed. |
When I build latest version v2-20-2, |
I have moved your issue (@jan-busa) into a separate one, because I this one is much broader and more work in general than solving yours (I hope) |
The cmake configuration of several ilcsoft packages is pretty old and could potentially profit from exploiting newer cmake functionalities and in general newer cmake paradigms, e.g. target based configurations. Since LCIO is at the very base of everything, it could serve as a good starting point to check what would need changing and could then serve as an example for other packages as well.
A few points that should be decided/considered beforehand:
XXX_LIBRARIES
,XXX_INCLUDE_DIRS
approach can co-exist (see, e.g. SIO, so that it should be possible to update this package by package without breaking everything while updating.A more modern cmake configuration has the potential to make some of the cmake macros that are part of ilcsoft obsolete, as cmake can now handle many things for which these macros were originally designed by itself. To which degree we can remove these macros is hard to estimate and will be something we find out along the way.
Other TODOs (keeping them here because then they show up as X of Y in the overview):
FindPython
to detect pythonBUILD_TESTING
option is considered for all testsThe text was updated successfully, but these errors were encountered: