spdlog icon indicating copy to clipboard operation
spdlog copied to clipboard

In fmt master, basic_format_string has moved from base.h to xchar.h

Open pratikpc opened this issue 1 year ago • 7 comments

Took place in this commit

This change causes compile time errors due to:-

  1. Using SPDLOG_FMT_EXTERNAL and using fmt master
  2. spdlog/common needing basic_format_string without SPDLOG_USE_STD_FORMAT https://github.com/gabime/spdlog/blob/e593f6695c6065e6b345fe2862f04a519ed484e0/include/spdlog/common.h#L367-L370
  3. xchar.h not being included due to absence of the specified definitions https://github.com/gabime/spdlog/blob/e593f6695c6065e6b345fe2862f04a519ed484e0/include/spdlog/common.h#L56-L58

pratikpc avatar Sep 26 '24 16:09 pratikpc

I solved it personally for myself by removing the ifdefs around the headers

pratikpc avatar Sep 26 '24 16:09 pratikpc

This is a change that has not yet been released, so we will figure out how to address it after its release.

tt4g avatar Sep 26 '24 21:09 tt4g

I run in exact the same issue -> https://github.com/gabime/spdlog/issues/3258

Vertexwahn avatar Nov 15 '24 21:11 Vertexwahn

Same issue here (wrongly addressed in https://github.com/gabime/spdlog/pull/3274)

ahoarau avatar Nov 30 '24 11:11 ahoarau

Hi @gabime , has this issue been fixed? Thanks.

martingalvan-volue avatar Nov 19 '25 11:11 martingalvan-volue

fixed many moons ago...

Vertexwahn avatar Nov 19 '25 13:11 Vertexwahn

fixed many moons ago...

Thanks. This ticket should then be closed (ideally with a link to the commit which fixed it).

martingalvan-volue avatar Nov 19 '25 13:11 martingalvan-volue