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

Use FairCMakeModules for find_package2 #1059

Merged
merged 4 commits into from
Apr 25, 2021

Conversation

ChristianTackeGSI
Copy link
Member

@ChristianTackeGSI ChristianTackeGSI commented Apr 22, 2021

  • Make cmake module installation more granular (only install specific subdirs of cmake/)
  • Optionally use FairCMakeModules for some modules. Fallback is in cmake/fallback/
  • Extract find_package into FairFindPackage2.cmake (added to fallback directory)

Checklist:

@ChristianTackeGSI ChristianTackeGSI marked this pull request as draft April 23, 2021 16:05
@ChristianTackeGSI ChristianTackeGSI changed the title Extract find_package2 into own File Use FaicCMakeModules for find_package2 Apr 25, 2021
CMakeLists.txt Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
@dennisklein dennisklein changed the title Use FaicCMakeModules for find_package2 Use FairCMakeModules for find_package2 Apr 25, 2021
* Move cmake module installation from main CMakeLists.txt
  to cmake/
* Only install specific directories from cmake/
* Fix FindFairRoot.cmake to be able to find this again
To make it easier to share find_package2 across projects
let's move it into its own file.
Optionally (highly recommended) use FairCMakeModules to
provide FairFindPackage2.
If FairCMakeModules isn't there, we currently have a
fallback mechanism.

Fallback sheduled for removal in 19.2.
@ChristianTackeGSI ChristianTackeGSI marked this pull request as ready for review April 25, 2021 16:06
@dennisklein dennisklein merged commit 641d001 into FairRootGroup:dev Apr 25, 2021
@ChristianTackeGSI ChristianTackeGSI deleted the pr/find_package2 branch April 26, 2021 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants