

<HTML><BODY><div>Hi, Igor!</div><div>Thanks for the patch!</div><div>LGTM.</div><div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Best regards,</div><div>Maxim Kokryashkin</div></div></div><div> </div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Вторник, 10 января 2023, 14:27 +03:00 от Igor Munkin <imun@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16733500230439638308_BODY">Sergey,<br><br>Thanks for your review!<br><br>On 10.01.23, Sergey Kaplun wrote:<br>> Hi, Igor!<br>><br>> Thanks for the patch!<br>> LGTM!<br><br>Added your tag:<br>| Reviewed-by: Sergey Kaplun <<a href="/compose?To=skaplun@tarantool.org">skaplun@tarantool.org</a>><br><br>><br><br><snipped><br><br>> > (*) Max, could you please check this patch on the Alpine?<br>><br>> Should we add the Alpine build and test in our CI in the scope of this<br>> commit or with separate one?<br><br>Honestly, I don't want to make LuaJIT CI similar to Tarantool one in<br>this context: if you commit to support the Linux-based distro zoo in our<br>CI, feel free to extend it by yourself. I'd rather introduce FreeBSD (or<br>another BSD distro) instead of another fancy Linux with little tweaks<br>that differ it from the popular ones.<br><br>><br><br><snipped><br><br>><br>> --<br>> Best regards,<br>> Sergey Kaplun<br><br>--<br>Best regards,<br>IM</div></div></div></div></blockquote><div> </div></BODY></HTML>