-
Notifications
You must be signed in to change notification settings - Fork 622
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Is the static still not working? #225
Comments
You can try with this vcpkg patch (also includes some physx stuff i'm working on): eddiejames/vcpkg@4202900 My original PR wasn't accepted because of webrtc stuff... I'll try and resubmit my change sometime |
I applied this patch, GNS compiled statically, but my project compiles with errors of unresolved external symbols.
I have enabled the use of static vcpkg libraries in the project properties. I don't know why vcpkg didn't include the library in my project. Can this be done manually? |
Okay, I solved this problem with a macro. #define STEAMNETWORKINGSOCKETS_STATIC_LINK But now there are the same errors in libcrypto.lib.
If I can solve this problem too, I'll write. |
I did it. Thank you for your patch. The problem I wrote about above is solved by using additional Windows API libraries. #pragma comment(lib, "Advapi32.lib")
#pragma comment(lib, "User32.lib")
#pragma comment(lib, "Crypt32.lib") It is also required to use the /MTd flag for Debug configuration and /MT for Release configuration. |
Also a little warning, for those who will be switching to the static version. If you call |
Static should be a preferred way to consume GameNetworkingSockets. Dll makes it easier for reverse engineers to hack your game. It would be nice if static linking was allowed with vcpkg. |
I am also blocked on proper support for VCPKG_LIBRARY_LINKAGE static. I hope it is coming soon. |
As far as I understood, in 1.4.0 you fixed the installation of the static version via vcpkg... or not?
I saw this PR [gamenetworkingsockets] Enable building as a static library, and add webrtc feature, it was not included in 1.4.0? Your PR 1.4.0 still does not support the static version. Why?
The text was updated successfully, but these errors were encountered: