The Wayback Machine - https://web.archive.org/web/20201210051746/https://github.com/catchorg/Catch2/issues/2106
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

Add "surrogate TUs" for all library headers #2106

Open
horenmar opened this issue Nov 19, 2020 · 1 comment
Open

Add "surrogate TUs" for all library headers #2106

horenmar opened this issue Nov 19, 2020 · 1 comment

Comments

@horenmar
Copy link
Member

@horenmar horenmar commented Nov 19, 2020

Surrogate TUs are TUs that only include one single header, and do nothing else. These TU must still include: if they do not, then a header doesn't stand on its own, and there is likely some include or declaration missing from it.

There are three parts to this

  • Add surrogate TUs for all current headers
  • Ensure that they are built during testing (probably under "Extra Tests" CMake toggle, to save compilation times for common builds)
  • Add CI script that checks whether all headers have their surrogate TU
@innerout
Copy link
Contributor

@innerout innerout commented Nov 25, 2020

I will be working on this issue. It seems like a good starting point to learn the codebase.

@horenmar horenmar assigned horenmar and innerout and unassigned horenmar Nov 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.