Produce different names for files in debug builds on MSVC

[jherico]

Fix for #310

[johnkslang]

Which file names are effected?

[jherico]

The lib file artifacts that are installed, specifically glslang.lib, HLSL.lib, OGLCompiler.lib, OSDependent.lib, and SPIRV.lib. With this PR, if you do a debug build in Visual Studio, each file gets a d suffix, like so glslangd.lib, HLSLd.lib, etc...

[jherico]

An alternative would be to modify the INSTALL targets so that the debug versions of the libraries are placed in a separate directory, but I don't know how to do that off the top of my head. using a d suffix is pretty common for this kind of thing in my experience.

[johnkslang]

Okay, so this will impact all downstream uses that link a debug executable based on the existing names.

[jherico]

Potentially, but as it stands, anyone using cmake to create and build the MSVC version of this project would presumably already be having issues since the debug and release versions overwrite one another in the install directory. However, if someone is already working around that issue or has another build pipeline that..

• uses MSVC
• requires the debug libs as well as the release libs
• has hardcoded the artifact names

Then yes they would have to update something.

[johnkslang]

So, downstream users that want debug libraries will need to modify their build system to pick them up. Otherwise, they'll get the non-debug libraries no matter how glslang was built. Yes? I think that's okay, just giving a fair warning time.