Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Use file(GENERATE) to create a CTestConfig file #7

Closed
bruxisma opened this issue Feb 5, 2019 · 1 comment
Closed

Use file(GENERATE) to create a CTestConfig file #7

bruxisma opened this issue Feb 5, 2019 · 1 comment
Assignees
Labels
critical 🚨 All hands on deck, this is a show stopper! ✨ enhancement ✨ New feature or request modules 🔮 Public API tools 🛠️ Tooling Support

Comments

@bruxisma
Copy link
Member

bruxisma commented Feb 5, 2019

Calling include(CTest) breaks a lot of stuff in what would be an otherwise normal build situation. One particular issue is that by calling enable_testing(), we can no longer make a custom target named test that also builds the unit test executables. In some places this build-and-run-tests target is called check, however we would like cmake --build check to be designated for clang-check. As such, we need to override all the CTest related machinery, including generating a CTestConfig file. We can do this via file(GENERATE), much like we would like to in #6. One additional feature we get out of this is the ability to
ignore ctest altogether. This will let us do things like hook into other project's unit test runners, or submit additional information to a code coverage service, such as codecov.

@bruxisma bruxisma added ✨ enhancement ✨ New feature or request modules 🔮 Public API tools 🛠️ Tooling Support labels Feb 5, 2019
@bruxisma bruxisma added this to the IXM v1.0 milestone Feb 5, 2019
@bruxisma bruxisma self-assigned this Feb 5, 2019
@bruxisma bruxisma added this to To do in 1.0 Release Feb 5, 2019
@bruxisma bruxisma added the critical 🚨 All hands on deck, this is a show stopper! label Feb 6, 2019
@bruxisma bruxisma modified the milestones: IXM v1.0, IXM Alpha Release Feb 6, 2019
@bruxisma bruxisma pinned this issue Feb 6, 2019
@bruxisma bruxisma added this to To do in Alpha Release Feb 6, 2019
@bruxisma bruxisma moved this from To do to In progress in Alpha Release Feb 7, 2019
@bruxisma bruxisma unpinned this issue Aug 29, 2019
@bruxisma
Copy link
Member Author

Closing, as this project has burned me out, and I cannot continue working on it in its current state. CMake has changed massively since this project was started. Most of it is no longer necessary. A new repository will be launched in September.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
critical 🚨 All hands on deck, this is a show stopper! ✨ enhancement ✨ New feature or request modules 🔮 Public API tools 🛠️ Tooling Support
Projects
No open projects
1.0 Release
  
To do
Alpha Release
  
In progress
Development

No branches or pull requests

1 participant