-
Notifications
You must be signed in to change notification settings - Fork 115
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
[bug] Runtime #130
Comments
The debug mode is covered here Lines 134 to 148 in 9dd7ae4
It has been tested in vcpkg for every combination static/dynamic debug/release and there was no problem there microsoft/vcpkg#10979 Could it be that it is "DEBUG" instead of "Debug"? |
Thank you simogasp for your feedback. |
Just to be sure, you are getting this when building popsift, not when building your project that links to popsift? With your Conan recipe, are you using ninja to build or are you generating the VS solution etc? BTW it's very nice and interesting that you are writing a conan recipe! |
Yes, it is when building popsfit project, and yes again with VS, not with ninja. |
It is actually broken when building from VS :-( Could you try to replace these lines from the cmake snippet above if(CMAKE_BUILD_TYPE STREQUAL "Debug")
set(PopSift_MVSC_LINKER "${PopSift_MVSC_LINKER}d")
endif()
list(APPEND CUDA_NVCC_FLAGS -Xcompiler ${PopSift_MVSC_LINKER}) with this ones if(CMAKE_BUILD_TYPE STREQUAL "Debug")
list(APPEND CUDA_NVCC_FLAGS_DEBUG -Xcompiler ${PopSift_MVSC_LINKER}d)
else()
list(APPEND CUDA_NVCC_FLAGS_RELEASE -Xcompiler ${PopSift_MVSC_LINKER})
endif() That should be a proper way to init the flags. |
Thank you, it does the job. We have also conan packages hosted on our public artifactory. To directly install popsift, here is the command: |
Describe the bug
Debug Multithread runtime library in debug (/MDd) mode replaced by the release one (/MD) which result in a link error.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
When building in debug mode, warnings inform that the Multithread runtime library in debug mode (/MDd) is substituted by the realase one (/MD). This results in a link error LNK2038 avoiding to build popsift in debug mode.
In Visual Studio, by forcing /MD in the project settings (C/C++- Code generation) in debug mode, the project builds.
Log
warning message:
error message:
Desktop
The text was updated successfully, but these errors were encountered: