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

Build failure with current fmt head branch #1941

Closed
jcelerier opened this issue May 13, 2021 · 6 comments
Closed

Build failure with current fmt head branch #1941

jcelerier opened this issue May 13, 2021 · 6 comments

Comments

@jcelerier
Copy link
Contributor

jcelerier commented May 13, 2021

Hello,
since fmt at commit fmtlib/fmt@16f2ef9

spdlog does not build, with the following error on win32 (latest MSVC):

spdlog\include\spdlog\common-inl.h(61,53): 
error C2664: 
'void fmt::v7::format_system_error(fmt::v7::detail::buffer<T> &,int,const char *) noexcept': 
cannot convert argument 3 from 'const std::string' to 'const char *'
          with
          [
              T=char
          ]

Not critical, but a heads-up :)

@jcelerier
Copy link
Contributor Author

(small heads-up to @vitaut also in case the issue is in fmt instead of spdlog?)

@FreezePhoenix
Copy link

Believe it's because of the fact that FMT changed a handful of things, and Spdlog's local copy of the FMT API isn't synced.

@vitaut
Copy link

vitaut commented May 14, 2021

This is one of the system_error-related changes made in preparation for the major release. The (obvious) fix is to add .c_str().

@FreezePhoenix
Copy link

Side note that using the latest releases of both libspdlog and libfmt results in some unresolved symbols when trying to use them.

@lgbaldoni
Copy link
Contributor

fmt 1.8.0 released and build failing.

@gabime
Copy link
Owner

gabime commented Jun 24, 2021

Fixed in latest commit

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

5 participants