-
Notifications
You must be signed in to change notification settings - Fork 253
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
We need loggers #105
Comments
GLOG? |
I think spdlog is better, yaLanTingLibs is using spdlog. |
I don't have preference for the specific loggers. |
logger is necessary |
现在排查问题有点棘手 |
那就用spdlog? |
Try to add support-logger branch |
可以,可以先弄个整体,其实用哪个logger library不是重点,可以先定义属于我们库自己的宏,到时候换宏后面的就行,重点应该是确立哪里应该打出日志,方便排查问题 |
单独搞个分支不太好;一般像这种feature的流程一般是:个人分支在 POC -> 设计 RFC -> 逐步将更改合入主线 |
logger可能一个人搞不定 |
这种 feature 多人合作反而会很麻烦;一步做到很好当然是比较困难,但没有必要一步做好;一般先有一个顶层设计之后就可以逐步的进行改造了,肯定是需要长期多次迭代才能做好的 |
OK |
怎么引入依赖呢?引入代码还是像gtest那种用外部依赖。我看yaLanTingLibs是引入的代码。 |
引入代码比较好吧,可以少用户很多事,放在 third_party 里就好 |
都行吧,cmake也可以fetch,直接从github上拉取代码,bazel也OK的 |
Search before asking
What happened + What you expected to happen
Currently async_simple lacks logger subsystem to log events. Although we don't want async_simple to depend too many things, we should try to make async_simple to depend some loggers conditionally.
Are you willing to submit a PR?
The text was updated successfully, but these errors were encountered: