commit 65ad768d8603671fc1085fe115019e72a595ced8
author Juan Ramos <juan@lunarg.com> Wed Mar 15 13:51:43 2023 -0600
committer Juan Ramos <114601453+juan-lunarg@users.noreply.github.com> Wed Mar 15 13:55:17 2023 -0600
tree f91f0366b75d2d4da826aa20d1e6c156dd604599
parent d732b2de303ce505169011d438178191136bfb00

cmake: Minor CMake cleanup

- Fix project name to be less confusing
- Better testing

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 6f671eb..860acd8 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -41,13 +41,11 @@
 endfunction()
 vlk_get_header_version()
 
-project(Vulkan-Headers LANGUAGES C VERSION ${VK_VERSION_STRING})
-
-message(DEBUG "${PROJECT_NAME} = ${PROJECT_VERSION}")
+project(VULKAN_HEADERS LANGUAGES C VERSION ${VK_VERSION_STRING})
 
 add_library(Vulkan-Headers INTERFACE)
-target_include_directories(Vulkan-Headers INTERFACE $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>)
 add_library(Vulkan::Headers ALIAS Vulkan-Headers)
+target_include_directories(Vulkan-Headers INTERFACE $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>)
 
 if (CMAKE_VERSION VERSION_LESS "3.21")
     # https://cmake.org/cmake/help/latest/variable/PROJECT_IS_TOP_LEVEL.html

tests/CMakeLists.txt

diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
index eab59fa..e24f3d2 100644
--- a/tests/CMakeLists.txt
+++ b/tests/CMakeLists.txt
@@ -18,6 +18,8 @@
 # NOTE: For us testing just means that these header files compile
 # with reasonable warnings.
 
+message(STATUS "${PROJECT_NAME} = ${PROJECT_VERSION}")
+
 set(CMAKE_C_STANDARD 99)
 set(CMAKE_C_STANDARD_REQUIRED ON)
 set(CMAKE_C_EXTENSIONS OFF)
@@ -38,3 +40,7 @@
 # vk_layer.h
 add_library(vk_layer MODULE vk_layer.c)
 target_link_libraries(vk_layer PRIVATE Vulkan::Headers)
+
+if (NOT TARGET Vulkan-Headers)
+    message(FATAL_ERROR "Backcompat for Vulkan-Headers target broken!")
+endif()

tests/find_package/CMakeLists.txt

diff --git a/tests/find_package/CMakeLists.txt b/tests/find_package/CMakeLists.txt
index f042bdc..d9f32a4 100644
--- a/tests/find_package/CMakeLists.txt
+++ b/tests/find_package/CMakeLists.txt
@@ -1,8 +1,8 @@
-cmake_minimum_required(VERSION 3.10.2)
+cmake_minimum_required(VERSION 3.14.2)
 
 project(TEST_VULKAN_HEADERS_FIND_PACKAGE_SUPPORT LANGUAGES C)
 
-find_package(VulkanHeaders REQUIRED CONFIG) 
+find_package(VulkanHeaders REQUIRED CONFIG)
 
 if (NOT TARGET Vulkan::Headers)
     message(FATAL_ERROR "Vulkan::Headers target not defined")
@@ -21,3 +21,12 @@
     message(FATAL_ERROR "VulkanHeaders_VERSION not defined!")
 endif()
 message(STATUS "VulkanHeaders_VERSION = ${VulkanHeaders_VERSION}")
+
+add_library(foobar STATIC)
+
+target_link_libraries(foobar PRIVATE Vulkan::Headers)
+
+target_sources(foobar PRIVATE
+    ../vk_icd.c
+    ../vk_layer.c
+)