search

sensics / OSVR-RenderManager

Apache License 2.0
64 stars 46 forks source link

Add header-dependency tests #114

Closed godbyk closed 8 years ago

godbyk commented 8 years ago

If BUILD_TESTS is enabled in CMake, this will run the same header-dependency tests that we use in OSVR-Core. Each test includes a single header file and builds a no-op program. If the program builds successfully, it means the header includes (directly or indirectly) all its dependencies. If it fails to build, it may be missing a dependency, or have some other program (check the compiler output for details).

Headers from submodules have not been tested yet, so you may encounter errors when building those. If they require libraries or should be ignored entirely, edit the tests/header_dependencies/CMakeLists.txt file appropriately.

russell-taylor commented 8 years ago

Sounds good. Once the conflicts are removed, I can test and pull.

godbyk commented 8 years ago

@russell-taylor I've rebased this against master so it should merge cleanly now.

russell-taylor commented 8 years ago

Well, the header tests turned up some issues. Working through them, trying to figure out whether to fix or ignore in each case.

godbyk commented 8 years ago

I figure it's usually better to have the automated build system yelling at you than and end user.