

<HTML><BODY><div class="cl-kfr6jq02di"><div>Hi, Sergey!</div><div>Thanks for review!<br>Fixed issues and force pushed the branch.</div> <div class="mail-quote-collapse"><blockquote style="border-left:1px solid #0857A6;margin:10px;padding:0 0 0 10px"><span>From: Sergey Kaplun <<a href="mailto:skaplun@tarantool.org">skaplun@tarantool.org</a>><br>To: Evgeniy Temirgaleev <<a href="mailto:e.temirgaleev@tarantool.org">e.temirgaleev@tarantool.org</a>><br>Cc: Sergey Bronnikov <<a href="mailto:sergeyb@tarantool.org">sergeyb@tarantool.org</a>>,<a href="mailto:tarantool-patches@dev.tarantool.org">tarantool-patches@dev.tarantool.org</a><br>Date: Monday, August 17, 2026 3:32 PM +03:00</span><br> <div><div id=""><div class="cl-488fm8fxkq"><div class="js-helper_mr_css_attr js-readmsg-msg_mr_css_attr"><div id="style_17869699620850206466_mr_css_attr"><div id="style_17869699620850206466_BODY_mr_css_attr">Hi, Evgeniy!<br>Thanks for the patch!<br>LGTM, with minor nits regarding the commit message.<br><br>On 08.08.26, Evgeniy Temirgaleev wrote:<br>> From: Temir Galeev <<a href="mailto:temir.galeev@bk.ru">temir.galeev@bk.ru</a>><br>><br>> The brew gcc installation is useless, because cmake use<br><br>Typo: s/use/uses/</div></div></div></div></div></div></blockquote></div></div><div>fixed</div><div class="cl-kfr6jq02di"><div class="mail-quote-collapse"><blockquote style="border-left:1px solid #0857A6;margin:10px;padding:0 0 0 10px"><div><div><div class="cl-488fm8fxkq"><div class="js-helper_mr_css_attr js-readmsg-msg_mr_css_attr"><div><div><br>> the clang compiler which is already present in the system.<br><br>Typo: s/compiler/compiler,/</div></div></div></div></div></div></blockquote></div></div><div>fixed</div><div class="cl-kfr6jq02di"><div class="mail-quote-collapse"><blockquote style="border-left:1px solid #0857A6;margin:10px;padding:0 0 0 10px"><div><div><div class="cl-488fm8fxkq"><div class="js-helper_mr_css_attr js-readmsg-msg_mr_css_attr"><div><div><br>><br>> Also, the clang is better than gcc for macOS.<br>><br>> Branch: <a href="https://github.com/tarantool/luajit/tree/tmr_g/lj-noticket-ci-build-mac">https://github.com/tarantool/luajit/tree/tmr_g/lj-noticket-ci-build-mac</a><br><br>Side note: It's better to insert the additional info under --- below to<br>avoid confusion with the commit message.</div></div></div></div></div></div></blockquote></div></div><div>Thanks! But here the note about clang is a part of the commit message.</div><div class="cl-kfr6jq02di"><div class="mail-quote-collapse"><blockquote style="border-left:1px solid #0857A6;margin:10px;padding:0 0 0 10px"><div><div><div class="cl-488fm8fxkq"><div class="js-helper_mr_css_attr js-readmsg-msg_mr_css_attr"><div><div><br>><br>> ---<br>> .github/actions/setup-macos/action.yml | 7 +++++--<br>> 1 file changed, 5 insertions(+), 2 deletions(-)<br>><br>> diff --git a/.github/actions/setup-macos/action.yml b/.github/actions/setup-macos/action.yml<br>> index 91632648..9e752a70 100644<br>> --- a/.github/actions/setup-macos/action.yml<br>> +++ b/.github/actions/setup-macos/action.yml<br><br><snipped><br><br>><br><br>--<br>Best regards,<br>Sergey Kaplun</div></div></div></div></div></div></blockquote></div></div><div><div>--<br>Best regards,</div><div>Evgeniy Temirgaleev</div></div></BODY></HTML>