Skip to content
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

Linker Warnings/Errors on Windows #1042

Open
renebckr opened this issue Sep 6, 2022 · 5 comments
Open

Linker Warnings/Errors on Windows #1042

renebckr opened this issue Sep 6, 2022 · 5 comments
Labels

Comments

@renebckr
Copy link

renebckr commented Sep 6, 2022

Description

After following the Guide in README.MD I tried compiling the generated solution. Initially, the Release configuration could not be linked properly. There were many errors concerning the default lib. Every other configuration also threw warnings for conflicts of default libs.
It seems like some dependencies use different default libraries.

The Warning: https://docs.microsoft.com/en-us/cpp/error-messages/tool-errors/linker-tools-warning-lnk4098?view=msvc-170

There are ignored default libraries for the Release configuration that caused the linker failure. Only after removing both ignored libraries, the application would launch properly. Also, the ignored libraries (libcmt.lib; libcmtd.lib) would indicate Multithreaded DLL run time usage. This conflicts with the static libs downloaded via vcpk.

Now my question is: Is this just me? Does it work properly for everybody else?

Additional Information

Used CMake command for the generation of the solution:
cmake -DVCPKG_TARGET_TRIPLET=x86-windows-static -G "Visual Studio 15 2017" --toolchain "G:/Projekte/openspades/vcpkg/scripts/buildsystems/vcpkg.cmake" -A Win32 -S "G:/Projekte/openspades" -B "G:/Projekte/openspades/OpenSpades.msvc"

Reproduction

  • Clone Repo
  • Follow Setup-Instructions for Windows
  • Open Solution using Visual Studio 2017
  • (Maybe required to switch to c++17 standard to compile as there are usages of "constexpr if")
  • Try compiling the release configuration
@yvt yvt added bug some feature is broken OS-Specific: Windows help-needed labels Sep 7, 2022
@PauliusV0
Copy link

For me I need libXext for some reason when it's a UNIX thing

@luis996
Copy link

luis996 commented Aug 23, 2023

I get the same issue

@luis996
Copy link

luis996 commented Aug 23, 2023

Bro this issue is a lot of old and it's still not fixed Bruhhh

@nonperforming
Copy link
Contributor

Readme is outdated. Check GitHub Actions file, it should build successfully

@renebckr
Copy link
Author

Readme is outdated. Check GitHub Actions file, it should build successfully

Interesting! It seems like it does a RelWithDebInfo build. Did anybody yet try to reproduce that workflow manually? It should be checked if that works for other build types as well. I would assume that we should be able to build Release/Debug-only builds as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants