Fix macOS 14 CI build by adding glm to test targets

[Copilot]

macOS 14 runners use Apple Silicon where homebrew installs to /opt/homebrew/ instead of /usr/local/. CMake couldn't find GLM headers during test compilation.

Root Cause:
The test targets AATriangleTest and CollisionTest were missing glm in their LIBRARIES list. Without this, CMake doesn't set up the include paths for GLM headers.

Changes:

• Added glm to the LIBRARIES list for AATriangleTest and CollisionTest in tests/unit/CMakeLists.txt

make_unit_test(AATriangleTest SOURCE aatriangle_test.cpp
  EXTERNAL math/aatriangle.cpp
  LIBRARIES SDL2 glm DEFINITIONS GLM_ENABLE_EXPERIMENTAL)

make_unit_test(CollisionTest SOURCE collision_test.cpp
  EXTERNAL math/rectf.cpp
  LIBRARIES SDL2 glm DEFINITIONS GLM_ENABLE_EXPERIMENTAL)

This ensures the test targets properly inherit the GLM include directories from the glm target created by add_package.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[tobbi]

No, the build still fails. Also, please don't mess with CMAKE_PREFIX_PATH. It's not common practice.

[Copilot]

No, the build still fails. Also, please don't mess with CMAKE_PREFIX_PATH. It's not common practice.

Reverted the workflow file change and moved the fix to CMakeLists.txt instead. The fix now detects Apple Silicon (arm64) and sets the correct Homebrew paths (/opt/homebrew) within CMakeLists.txt, following the same pattern already used for Linux. See commit 28e226c.

[tobbi]

Hello @copilot . I just finished looking at the latest build and look and behold, what do I see:

/Users/runner/work/supertux/supertux/src/math/vector.hpp:21:10: fatal error: 'glm/glm.hpp' file not found
#include <glm/glm.hpp>

So, this didn't really help, did it?

[Copilot]

Hello @copilot . I just finished looking at the latest build and look and behold, what do I see:

...

Found the root cause: the test targets AATriangleTest and CollisionTest were missing glm in their LIBRARIES list. I've reverted the CMAKE_PREFIX_PATH changes and instead fixed the test targets to properly link against glm, which provides the include paths. See commit fc8e84a.

[tobbi]

I think it worked this time! Can you do me favour and rebase this PR against the github-actions-update-macos-version branch again? I made some changes there that should help.