build: don't include fiber.h into a merger test
The problem is described in a previous commit. In short: inclusion of `fiber.h` leads to inclusion of `libunwind.h`, but it is not necessarily in include paths that used for testing sources. We can add the include path or eliminate the `fiber.h` dependency. I have no strict preference here. However the latter allows to write public API and internal API calls differently in the testing code and it looks more clean. The implementation is based on the `tnt_internal_symbol()` function, which was introduced to make a dark magic of this kind. See commit 395c30e8 ("fiber_channel: add accessor to internal functions"). Fixes #8025 NO_DOC=It is a fix of a build failure. NO_TEST=It is a fix of a build failure.
Showing
- changelogs/unreleased/fix-libunwind-h-search.md 3 additions, 0 deletionschangelogs/unreleased/fix-libunwind-h-search.md
- src/lib/core/fiber.c 6 additions, 0 deletionssrc/lib/core/fiber.c
- src/lib/core/fiber.h 6 additions, 0 deletionssrc/lib/core/fiber.h
- src/symbols.c 8 additions, 0 deletionssrc/symbols.c
- test/app-luatest/tnt_internal_symbol_test.lua 1 addition, 0 deletionstest/app-luatest/tnt_internal_symbol_test.lua
- test/box-tap/CMakeLists.txt 0 additions, 3 deletionstest/box-tap/CMakeLists.txt
- test/box-tap/check_merge_source.c 22 additions, 6 deletionstest/box-tap/check_merge_source.c
Loading
Please register or sign in to comment