search

fair-acc / gr-digitizers

GNU General Public License v3.0
3 stars 3 forks source link

cmake: only add ui tools and their dependencies in toplevel build #137

Closed wirew0rm closed 10 months ago

wirew0rm commented 10 months ago

When adding gr-digitizers to a build using add_subdirectory or fetch_content, the targets for the ui tools and dependencies can clash. This commit removes them from the build if this project is not the top-level cmake project.

Also default disables gr-digitizer tests in that case by default.

Longer term we should improve the cmake in all projects to allow overriding dependencies and correctly use cmake namespaces for the targets...

sonarcloud[bot] commented 10 months ago

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud