Skip to content

Commit

Permalink
Update tracetooltests
Browse files Browse the repository at this point in the history
We now source Vulkan-Headers from tracetooltests, avoiding
the possibility of a mismatch between the headers of it and
us.
  • Loading branch information
per-mathisen-arm committed Nov 6, 2024
1 parent a44abf8 commit b6edf2a
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 9 deletions.
3 changes: 0 additions & 3 deletions .gitmodules
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
[submodule "external/density"]
path = external/density
url = http://github.com/centaurean/density
[submodule "external/Vulkan-Headers"]
path = external/Vulkan-Headers
url = https://github.com/KhronosGroup/Vulkan-Headers.git
[submodule "tracetooltests"]
path = tracetooltests
url = https://github.com/ARM-software/tracetooltests.git
Expand Down
8 changes: 4 additions & 4 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ else()
message(STATUS "XCB and X11 support not added")
endif()

if(NOT EXISTS "${CMAKE_SOURCE_DIR}/external/Vulkan-Headers/.git")
if(NOT EXISTS "${CMAKE_SOURCE_DIR}/external/tracetooltests/external/Vulkan-Headers/.git")
message(SEND_ERROR "git submodules not available. Please run: git submodule update --init --recursive")
endif()

Expand Down Expand Up @@ -116,7 +116,7 @@ set(COMMON_INCLUDE
PUBLIC ${SRC_ROOT}/generated
PUBLIC ${SRC_ROOT}/external
PUBLIC ${SRC_ROOT}/external/tracetooltests/include
PUBLIC ${SRC_ROOT}/external/Vulkan-Headers/include
PUBLIC ${SRC_ROOT}/external/tracetooltests/external/Vulkan-Headers/include
PRIVATE ${SRC_ROOT}/external/jsoncpp
)

Expand Down Expand Up @@ -189,8 +189,8 @@ add_custom_command(
COMMAND ${PYTHON_EXECUTABLE} ${SRC_ROOT}/scripts/tostring.py
COMMAND touch .autogenerated
WORKING_DIRECTORY ${SRC_ROOT}
DEPENDS ${SRC_ROOT}/scripts/lava.py ${SRC_ROOT}/external/Vulkan-Headers/registry/vk.xml ${SRC_ROOT}/external/tracetooltests/scripts/spec.py ${SRC_ROOT}/src/hardcode_read.cpp
${SRC_ROOT}/src/hardcode_write.cpp ${SRC_ROOT}/scripts/util.py ${SRC_ROOT}/external/Vulkan-Headers/include/vulkan/vk_layer.h ${SRC_ROOT}/scripts/vk_wrapper.py
DEPENDS ${SRC_ROOT}/scripts/lava.py ${SRC_ROOT}/external/tracetooltests/external/Vulkan-Headers/registry/vk.xml ${SRC_ROOT}/external/tracetooltests/scripts/spec.py
${SRC_ROOT}/src/hardcode_read.cpp ${SRC_ROOT}/src/hardcode_write.cpp ${SRC_ROOT}/scripts/util.py ${SRC_ROOT}/scripts/vk_wrapper.py
)

# The below is necessary because cmake is too braindead to realize we should not run the above once per target
Expand Down
1 change: 0 additions & 1 deletion external/Vulkan-Headers
Submodule Vulkan-Headers deleted from cfebfc

0 comments on commit b6edf2a

Please sign in to comment.