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

Error when enabling KOMPUTE_ENABLE_SPDLOG #268

Closed
david-cortes opened this issue Feb 10, 2022 · 3 comments
Closed

Error when enabling KOMPUTE_ENABLE_SPDLOG #268

david-cortes opened this issue Feb 10, 2022 · 3 comments

Comments

@david-cortes
Copy link

Using #define KOMPUTE_ENABLE_SPDLOG (as is suggested by the ifdef's in the examples) triggers a compilation error:

kompute/Kompute.hpp:640:26: error: #if with no expression
  640 | #if KOMPUTE_ENABLE_SPDLOG

This is regardles of whether #include <spdlog/spdlog.h> is added before or not.

@axsaucedo
Copy link
Member

How are you building the project? By default a value is provided to KOMPUTE_ENABLE_SPDLOG by cmake, so it should still work, are you looking to disable it? If that's the case you just have to add KOMPUTE_ENABLE_SPDLOG=0 - could you try that?

@david-cortes
Copy link
Author

Oh, I was thinking this was a configuration option for projects that link to this library rather than for the library itself.

@axsaucedo
Copy link
Member

No worries @david-cortes - this is certainly an external variable for users to configure accordingly, but the main thing is that it has to be set to a value otherwise it would error out as you suggest - I think perhaps adding a further note in the docs could provide more context

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

No branches or pull requests

2 participants