steam-audio icon indicating copy to clipboard operation
steam-audio copied to clipboard

Published 20 hours ago • verified publisher icon ValveSoftware

Don't link delayimp on UWP because it doesn't exist there

Open Ravbug opened this issue 9 months ago • 4 comments

delayimp.lib does not exist on UWP, but SteamAudio always tries to link it on Windows platforms. This PR prevents SteamAudio from linking delayimp.lib on UWP. SteamAudio still works on UWP without it.

Ravbug avatar Apr 27 '24 19:04 Ravbug

Did you need to make any other changes in order to build for UWP? I tried building your branch for UWP and it didn't seem to work.

lakulish avatar May 06 '24 21:05 lakulish

What error did you get when trying to build UWP?

Ravbug avatar May 16 '24 21:05 Ravbug

I got the following error:

error MSB8024: Using static version of the C++ runtime library is not supported.

Which can be fixed by removing the /MT flag set in the root CMakeLists.txt, but it would also require all dependencies to be built without static linking to the CRT, which would involve changing the dependency building scripts as well.

lakulish avatar May 16 '24 22:05 lakulish

Which can be fixed by removing the /MT flag set in the root CMakeLists.txt

I checked my copy and found that I also did that. My project builds SteamAudio's dependencies as part of the cmake build tree. Maybe an option could be added to disable adding the /MT flag?

Ravbug avatar May 20 '24 17:05 Ravbug