fix(build): issues with GLM
Description
We face issues when including GLM in Sight. Before the commit 498920bb, the dependency was not forwarded properly by a public interface. However, because of a buggy glmConfig-version.cmake
provided upstream, this breaks the configuration because find_package(GLM...)
returns an error if called twice. This should be fixed by a new patch in Debian but this will take time until it is available in Ubuntu LTS.
Steps to reproduce
Build a dependent repository.
Functional specifications
We just want the build to succeed without pain...
Technical specifications
Use a different approach depending on the platform:
- Use the old-ish system with
target_include_directories
on Linux - Use
glmConfig.cmake
on Windows (but everywhere this time)
Test plan
Build a dependent repository. Using find_package(GLM...)
twice in a row should not break.