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

build: Public Header Relocatability, main branch (2024.06.18.) #3303

Merged

Conversation

krasznaa
Copy link
Member

Made all libraries with public headers use CMAKE_INSTALL_INCLUDEDIR. They were already using that variable consistently for installing the headers. This is just to fix the declarations in the auto-generated CMake files of the Acts installations.

This is the same sort of update as https://gitlab.cern.ch/GeoModelDev/GeoModel/-/merge_requests/326. As we discussed during https://indico.cern.ch/event/1422046/, putting the headers of the different ATLAS externals into separate directories will help with making some specific builds of ours more robust. (Ones where we want to override the Acts version used in a nightly, with one built locally.)

I just got rid of $<INSTALL_INTERFACE completely from the FRNN code. 🤔 Since that code does not install any public headers, that was just there to confuse people. 😛

They were already using that variable consistently for installing
the headers. This is just to fix the declarations in the auto-generated
CMake files of the Acts installations.
@github-actions github-actions bot added Component - Fatras Affects the Fatras module Component - Plugins Affects one or more Plugins labels Jun 18, 2024
Copy link

codecov bot commented Jun 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.31%. Comparing base (eb8848f) to head (f9685e4).
Report is 37 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3303   +/-   ##
=======================================
  Coverage   47.31%   47.31%           
=======================================
  Files         512      512           
  Lines       30440    30440           
  Branches    14795    14795           
=======================================
  Hits        14403    14403           
  Misses       5404     5404           
  Partials    10633    10633           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@paulgessinger paulgessinger changed the title build, fix: Public Header Relocatability, main branch (2024.06.18.) build: Public Header Relocatability, main branch (2024.06.18.) Jul 3, 2024
@paulgessinger paulgessinger added this to the v36.0.0 milestone Jul 3, 2024
Copy link

sonarcloud bot commented Jul 3, 2024

@kodiakhq kodiakhq bot merged commit d89cb7d into acts-project:main Jul 3, 2024
51 checks passed
@github-actions github-actions bot removed the automerge label Jul 3, 2024
@acts-project-service acts-project-service added Breaks Athena build This PR breaks the Athena build Fails Athena tests This PR causes a failure in the Athena tests labels Jul 4, 2024
paulgessinger added a commit to paulgessinger/acts that referenced this pull request Jul 6, 2024
paulgessinger added a commit to paulgessinger/acts that referenced this pull request Jul 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaks Athena build This PR breaks the Athena build Component - Fatras Affects the Fatras module Component - Plugins Affects one or more Plugins Fails Athena tests This PR causes a failure in the Athena tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants