Jack·Boos·Yu
Jack·Boos·Yu
Hi everyone, does this issue still exists?
The reason for reporting this error is that the library was generated incorrectly. We expected to use dynamic CRT linkage but static CRT linkage. vcpkg uses the Visual Studio project...
@mattbucci The patch `0001-fix-crt-linking.patch` fixes the CRT linkage, ~so if you want to build static, that patch should be applied.~ This patch modifies the CRT linage under static configuration to...
If the latest version does not work properly, please leave me the logs under _VCPKG_ROOT/buildtrees/mpg123_. Thanks.
@mattbucci I think so. Thanks for this info, I'll digger into it later.
Confirmed. And you should use community triplet `x86-windows-static-md` instead of change triplet file `x86-windows`.
@superhero1 This is easy to solve, just comment out _VCPKG_ROOT/ports/mpg123/porfile.cmake_ line 33: `0001-fix-crt-linking.patch` will solve this issue. Then use command `./vcpkg install mpg123:x86-windows-static-md`. Build okay.
The reason for this issue is that the patch failed to patch the RuntimeLibrary option in `vcxproj` according to `VCPKG_CRT_LINKAGE`. I will open a PR to fix the issue.
@harrishancock I've already moved it to https://github.com/capnproto/capnproto/blob/master/doc/install.md , can you review it again? Thanks.
@kentonv Since vcpkg is a package manager that supports multiple platforms(including Windows, Linux and OSX), I don't think it is a good choice to put it on a certain platform.