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

Getting compile-error in logger_impl.h#L255 #571

Closed
hestad opened this issue Dec 6, 2017 · 12 comments
Closed

Getting compile-error in logger_impl.h#L255 #571

hestad opened this issue Dec 6, 2017 · 12 comments

Comments

@hestad
Copy link
Contributor

hestad commented Dec 6, 2017

There is an extra right curly braces. (at least for clang 5)

How come your build-server flagged this as green?

@gabime
Copy link
Owner

gabime commented Dec 6, 2017

Because the clang tests were removed a while ago :(

@gabime
Copy link
Owner

gabime commented Dec 6, 2017

Fixed in 88fe218

Please verify it solves your problem

@hestad
Copy link
Contributor Author

hestad commented Dec 6, 2017

I would guess you never call #define SPDLOG_WCHAR_TO_UTF8_SUPPORT in your tests?

@hestad
Copy link
Contributor Author

hestad commented Dec 6, 2017

Yes, that fixes it for me.

@hestad
Copy link
Contributor Author

hestad commented Dec 6, 2017

Btw, since I have you here :) when do you plan to release spdlog next time?

@gabime
Copy link
Owner

gabime commented Dec 6, 2017

Very soon. Hopefully in the next few days. Why?

@hestad
Copy link
Contributor Author

hestad commented Dec 6, 2017

I'm waiting for two of the wstring-fixes, but also a fmt-fix related to wstring. (ref:
fmtlib/fmt#606 (comment) )

I have just copied the sources for now, but ideally we would get spdlog + fmt via conan.

@gabime
Copy link
Owner

gabime commented Dec 6, 2017

Please note that spdlog bundles only official fmt versions, not necessarily with the latest fixes from the fmt repo..

@hestad
Copy link
Contributor Author

hestad commented Dec 6, 2017

Yeah, I guessed as much, but I also agree. I had a tiny dream that they could be co-released :)

@gabime
Copy link
Owner

gabime commented Dec 6, 2017

Do you know when fmt plan to release? that can be arranged

@hestad
Copy link
Contributor Author

hestad commented Dec 7, 2017

Seems like they accepted to do a release the next few days: fmtlib/fmt#621 (comment)

@gabime
Copy link
Owner

gabime commented Dec 7, 2017

Oh thats good. I will wait..

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