

<HTML><BODY><div>Thanks for the review, Sergey!</div><div> </div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div> <blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16273803260886262874_BODY">Hi, Maxim!<br><br>Thanks for the patch!<br><br>Please consider my comments below.<br><br>On 20.07.21, Maxim Kokryashkin wrote:<br>> Backports 'supoort Tarantool' patch[1] from lua-Harness.<br>><br>> [1]: <a href="https://framagit.org/fperrad/lua-Harness/-/commit/d3ceee16" target="_blank">https://framagit.org/fperrad/lua-Harness/-/commit/d3ceee16</a><br>><br>> Part of tarantool/tarantool#5970<br>> Part of tarantool/tarantool#4473<br>> ---<br><br>| test: support tarantool in lua-Harness<br><br>Nit: s/tarantool/Tarantool/.<br><br>|<br>| Backports 'support Tarantool' patch[1] from lua-Harness.<br><br>Nit: s/Tarantool/tarantool/ regarding to the original commit message.<br><br>| This patch introduces lua-Harness profile for Tarantool considering the<br>| platform specifics. Furthermore, these changes skip all CLI-related<br>| tests for Tarantool binary, so 241-standalone.t and 411-luajit.t can be<br>| renamed back.<br><br>I afraid, not so fast. Within these changes tests are failed in the<br>LuaJIT's repo with the following error:<br>|! unknown luaJIT command or jit.* modules not installed<br><br>Seems like LUA_PATH should be adjusted for these tests.<br>|<br>| [1]: <a href="https://framagit.org/fperrad/lua-Harness/-/commit/d3ceee16" target="_blank">https://framagit.org/fperrad/lua-Harness/-/commit/d3ceee16</a><br>|<br>| Part of tarantool/tarantool#5970<br>| Part of tarantool/tarantool#4473<br><br>> ...standalone.t.disabled => 241-standalone.t} | 5 ++<br>> test/lua-Harness-tests/320-stdin.t | 2 +-<br>> .../{411-luajit.t.disabled => 411-luajit.t} | 2 +-<br>> test/lua-Harness-tests/profile_tarantool.lua | 63 +++++++++++++++++++<br>> 4 files changed, 70 insertions(+), 2 deletions(-)<br>> rename test/lua-Harness-tests/{241-standalone.t.disabled => 241-standalone.t} (99%)<br>> rename test/lua-Harness-tests/{411-luajit.t.disabled => 411-luajit.t} (99%)<br>> create mode 100644 test/lua-Harness-tests/profile_tarantool.lua<br><br><snipped><br><br>--<br>Best regards,<br>Sergey Kaplun</div></div></div></div></blockquote><div> </div></div></blockquote></BODY></HTML>