forked from ruslo/hunter
-
Notifications
You must be signed in to change notification settings - Fork 173
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
[Regression] spdlog fails to compile in v0.23.322 #518
Labels
bug
Something isn't working
Comments
Uh oh! I just added cpp-pm-packages to that repo. |
Thalhammer
added a commit
to Thalhammer/hunter
that referenced
this issue
Feb 10, 2022
rbsheth
pushed a commit
that referenced
this issue
Feb 10, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I've read Brief overview section and do understand basic concepts. [Yes]
I've read F.A.Q. section and there is no solution to my problem there. [Yes]
I've read Code of Conduct, I promise to be polite and will do my best at being constructive. [Yes]
I've read Reporting bugs section carefully. [Yes]
I've checked that all the
hunter_add_package
/find_package
API used by me in the example is the same as in documentation. [Yes]I'm using latest Hunter
URL
/SHA1
. [Yes]I've created SSCCE reproducing the issue:
CMakeLists.txt
Here is the log until first error reported by Hunter, option
HUNTER_STATUS_DEBUG
isON
:CMake Log
I've checked that the first error in logs IS NOT
external.build.failed
. [Yes]I'm using the next command line on generate step:
This might be related to #514 and gabime/spdlog#1975.
Reverting
fmt
to version 7.1.2 using a custom config.cmake fixes the issue, but this is obviously not a scalable solution. We probably need to update spdlog to a newer version (it's outdated anyway), but I think there should be some way to check against this.Some examples of what could be done:
I just wanted to do the update myself and noticed that I can't push to cpp-pm/spdlog because for some reason its not part of the cpp-pm-packages team. Probably just an oversight but would be cool if you could add it @rbsheth .
The text was updated successfully, but these errors were encountered: