Made libspirv include relative to allow better portability

KhronosGroup / SPIRV-Tools

Apache License 2.0

1.09k stars 559 forks source link

Made libspirv include relative to allow better portability #5807

Closed Nielsbishere closed 2 months ago

Nielsbishere commented 2 months ago

This can produce issues when vulkan sdk is installed (I'm encountering an issue on mac). This is because it might fetch spirv-tools from the vulkan sdk which may or may not be the right version.

CLAassistant commented 2 months ago

All committers have signed the CLA.

Nielsbishere commented 2 months ago

@dneto0 what would be the problem it could introduce? Won't local files relative to the current file always take priority over any other include directories?