-
Notifications
You must be signed in to change notification settings - Fork 376
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: add CMake configuration for evread module #9853
build: add CMake configuration for evread module #9853
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, Maxim!
Thanks for the patch!
LGTM, except a general nit to all changes.
Please sort all new entries alphabetically.
200a6cb
to
597d408
Compare
Thanks for the comment! Fixed for places where order is insignificant. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would replace LJ
with something more clear, for example "covered by LuaJIT tests":
NO_DOC=LJ
NO_TEST=LJ
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maxim, thanks for the patch! LGTM with minor comments.
This patch adds necessary cmake configurations for the <evread.lua> module, so it can be used later to implement human-readable error reporting in profile parsers. Part of tarantool#9217 NO_DOC=LuaJIT submodule NO_TEST=covered by the LuaJIT tests NO_CHANGELOG=build
597d408
to
84101f0
Compare
Fixed, thanks |
This patch adds necessary cmake configurations for the <evread.lua> module, so it can be used later to implement human-readable error reporting in profile parsers.
Part of #9217
NO_DOC=LJ
NO_TEST=LJ
NO_CHANGELOG=build