fix: use the current build config for vulkan-shaders-gen

[giladgd]

This solves an issue where when building on Windows on Arm with a Release config, vulkan-shaders-gen is built with Debug config (since no config is set), so the built binary is placed on an output folder that cmake doesn't expect, which then fails to run the built vulkan-shaders-gen.

[bandoti]

Instead of passing these arguments as a build switch, I am wondering if we should instead add more flags to CMAKE_ARGS to control these things at generation time. For example:

# Pass additional important variables
if(CMAKE_CONFIGURATION_TYPES)
    # Multi-configuration generator case (Visual Studio, Xcode)
    list(APPEND VULKAN_SHADER_GEN_CMAKE_ARGS
        -DCMAKE_CONFIGURATION_TYPES="${CMAKE_CONFIGURATION_TYPES}")
else()
    # Single-configuration generator case (Makefile, Ninja)
    if(CMAKE_BUILD_TYPE)
        list(APPEND VULKAN_SHADER_GEN_CMAKE_ARGS
            -DCMAKE_BUILD_TYPE=${CMAKE_BUILD_TYPE})
    endif()
endif()

[giladgd]

Doing that way my first try, but it didn't seem to affect the output folder.
With the current build type being Release and a -DCMAKE_BUILD_TYPE=Release flag in VULKAN_SHADER_GEN_CMAKE_ARGS, the output was still being put in a Debug directory.

BTW do we use the vulkan-shaders-gen binary after using it once here?
If we don't need it afterwards, it may be a good idea to remove it later from the output directory.

[bandoti]

Ah okay, well if you tried it already and the current change is working then I'm happy to approve the change as-is.

We want to keep that binary in there in case one wants to run it afterward manually I think. If it's not impeding anything to keep it around it should be fine.