CMake is looking for `../Vulkan-Headers/`

icculus commented 3 years ago

Can we look for these in a standard place, or just include them with the MojoShader sources?

flibitijibibo commented 3 years ago

Will page @thatcosmonaut in case he's aware of any known trick, but the source of this is us simply including Vulkan-Headers in the parent project:

Bundling the headers directly can cause conflicts with the renderer that uses MojoShader, and unlike GL/ the headers for Vulkan are... kind of huge.

I would suggest:

Start with ../Vulkan-Headers for local builds
If it doesn't exist, search for system headers (I want to say find_package but that function is awful and probably does a search for libvulkan.so too)
Error if neither exist

thatcosmonaut commented 3 years ago

I agree with that strategy.

flibitijibibo commented 2 years ago

icculus / mojoshader