Skip to content

Conversation

@charles-lunarg
Copy link
Contributor

@charles-lunarg charles-lunarg commented Sep 17, 2025

This PR does two things: Codegen the 'current generated version' into CMake's project() field, and use that version to specify the 'minimum VulkanHeader's version required' in find_package. This turns the cryptic compile errors from compiling with too-old Vulkan-Headers into a clear build failure with a specific error message.

Example error message

CMake Error at CMakeLists.txt:55 (find_package):
  Could not find a configuration file for package "VulkanHeaders" that is
  compatible with requested version "1.4.326".
  
  The following configuration files were considered but not accepted:
  /home/cdgiessen/lunarg/Vulkan-Loader/external/Debug/64/Vulkan-Headers/build/install/share/cmake/VulkanHeaders/VulkanHeadersConfig.cmake, version: 1.4.325

@ci-tester-lunarg
Copy link

CI Vulkan-Tools build queued with queue ID 532281.

@ci-tester-lunarg
Copy link

CI Vulkan-Tools build queued with queue ID 532296.

@ci-tester-lunarg
Copy link

CI Vulkan-Tools build # 1808 running.

@ci-tester-lunarg
Copy link

CI Vulkan-Tools build # 1808 passed.

@charles-lunarg charles-lunarg force-pushed the version_check_vulkan_headers branch from b3885a9 to f7702ef Compare September 17, 2025 16:15
@ci-tester-lunarg
Copy link

CI Vulkan-Tools build queued with queue ID 532877.

@ci-tester-lunarg
Copy link

CI Vulkan-Tools build queued with queue ID 532892.

@ci-tester-lunarg
Copy link

CI Vulkan-Tools build # 1810 running.

@ci-tester-lunarg
Copy link

CI Vulkan-Tools build # 1810 passed.

@charles-lunarg charles-lunarg merged commit 7663eb7 into KhronosGroup:main Sep 17, 2025
18 checks passed
@charles-lunarg charles-lunarg deleted the version_check_vulkan_headers branch September 17, 2025 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants