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

SPDLOG recent commit is propagating warnings #16153

Closed
prateek9623 opened this issue Feb 9, 2021 · 4 comments
Closed

SPDLOG recent commit is propagating warnings #16153

prateek9623 opened this issue Feb 9, 2021 · 4 comments
Assignees
Labels
category:port-bug The issue is with a library, which is something the port should already support

Comments

@prateek9623
Copy link
Contributor

Describe the bug
commit 8a95605 is propagating warning /wd4251 /wd4275 to our CUDA based application. Which doesn't go well with warning flags. Is there anything that can be done about this?

Environment

  • OS: Windows
  • Compiler: MSVC(14.28.29333), NVCC 11.2

To Reproduce
link with CUDA based application

Expected behavior
the warning should not propagate from spdlog

@NancyLi1013
Copy link
Contributor

Hi @prateek9623

Thanks for posting this issue.

The warning is from upstream, there is no way to solve this. Please see

    if(MSVC)
        target_compile_options(spdlog PUBLIC /wd4251 /wd4275)
    endif()

https://github.com/gabime/spdlog/blob/v1.x/CMakeLists.txt#L134-L136

You can port an issue to upstream if this blocks your application.

@NancyLi1013 NancyLi1013 added the category:question This issue is a question label Feb 10, 2021
@prateek9623
Copy link
Contributor Author

Hi @prateek9623

Thanks for posting this issue.

The warning is from upstream, there is no way to solve this. Please see

    if(MSVC)
        target_compile_options(spdlog PUBLIC /wd4251 /wd4275)
    endif()

https://github.com/gabime/spdlog/blob/v1.x/CMakeLists.txt#L134-L136

You can port an issue to upstream if this blocks your application.

cant we create a patch to rectify this

@prateek9623
Copy link
Contributor Author

recent commit in spdlog: gabime/spdlog@8fbc853 fixes this issue

@NancyLi1013
Copy link
Contributor

Thanks for your kindly reminder @prateek9623. I will use the patch to fix it later.

@NancyLi1013 NancyLi1013 added category:port-bug The issue is with a library, which is something the port should already support and removed category:question This issue is a question labels Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants