Look for vulkan in CMakeLists

[iche033]

Signed-off-by: Ian Chen [email protected]

🦟 Bug fix

Summary

Make sure vulkan exists before building ogre2. This is needed by Jammy after #706 introduced the vulkan/vulkan_core.h include header. Otherwise build fails.

We should only just need the vulkan headers and not the whole library. Alternative solution is to manually search for vulkan_core.h using FIND_PATH in cmake. I went with gz_find_package for simplicity as the headers are installed with the library when you do apt install libvulkan-dev

• Update: ended up using find_path to look for vulkan headers and adding more logic to make sure ogre-next is built with vulkan support

Note that the libvulkan-dev dependency has already been added to packages-jammy.apt and our Jammy debbuild CI script.

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

[codecov]

Codecov Report

Merging #782 (a06f079) into main (4762f85) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #782   +/-   ##
=======================================
  Coverage   76.48%   76.48%           
=======================================
  Files         167      167           
  Lines       15099    15099           
=======================================
  Hits        11548    11548           
  Misses       3551     3551           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.