

<HTML><BODY><div>diff:</div><div> <div> </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_16273026770892817228_BODY"><div class="cl_304345"><div> <blockquote style="border-left:1px solid #0857A6;margin:10px;padding:0 0 0 10px;"><div id=""><div class="js-helper_mr_css_attr js-readmsg-msg_mr_css_attr"><div><div id="style_16272492722021221321_BODY_mr_css_attr">Max,<br><br>Thanks for the patch! Please consider the comments below.<br><br>At first, commit subject exceeds 50 symbols and the message exceeds 72<br>symbols. Please check this patch against our guidelines[1].<br><br>On 20.07.21, Maxim Kokryashkin wrote:<br>> Tarantool interactive mode misbehaviour has been found on FreeBSD (for more info see #6231).<br><br>The issue is mentioned the wrong way. The right format is<br>tarantool/tarantool#6231.<br><br>> Hence, this particular assertion is masked for FreeBSD until the mentioned issue resolves.<br><br>Strictly saying, you skipped all assertions (not only those using broken<br>interactive mode), not the particular one.<br><br>><br>> Resolves #5970<br><br>This patch doesn't resolve the issue, since Tarantool profile introduced<br>in "[PATCH v2 06/10] test: support tarantool in lua-Harness" is not used<br>by Tarantool CI. Hence this is only "Part of" patch.<br><br>> Part of #4473<br><br>BTW, both issues are also mentioned the wrong way. The right format is<br>tarantool/tarantool#<issue-number> for both cases (see the previous<br>patches in the series for the examples).<br><br>It's also worth to add "Relates to tarantool/tarantool#6231".<br><br>> ---<br>> Additional info: <a href="https://github.com/tarantool/tarantool/issues/5970#issuecomment-883253400" target="_blank">https://github.com/tarantool/tarantool/issues/5970#issuecomment-883253400</a><br>><br>> test/lua-Harness-tests/241-standalone.t | 4 ++++<br>> 1 file changed, 4 insertions(+)<br>><br>> diff --git a/test/lua-Harness-tests/241-standalone.t b/test/lua-Harness-tests/241-standalone.t<br>> index afbcf5b8..d5373b9f 100755<br>> --- a/test/lua-Harness-tests/241-standalone.t<br>> +++ b/test/lua-Harness-tests/241-standalone.t<br>> @@ -51,6 +51,10 @@ if not pcall(io.popen, lua .. [[ -e "a=1"]]) then<br>> skip_all "io.popen not supported"<br>> end<br>><br>> +if jit.os == 'BSD' then<br>> + skip_all "BSD is not supported yet"<br>> +end<br><br>At first, please add a comment with the rationale for this change.<br>Furthermore, it's better to skip only assertions using interactive mode<br>and run other assertions related to CLI behaviour.<br><br>> +<br>> plan'no_plan'<br>> diag(lua)<br>><br>> --<br>> 2.32.0<br>><br><br>[1]: <a href="https://www.tarantool.io/en/doc/latest/dev_guide/developer_guidelines/" target="_blank">https://www.tarantool.io/en/doc/latest/dev_guide/developer_guidelines/</a><br><br>--<br>Best regards,<br>IM</div></div></div></div></blockquote><div> </div></div></div></div></div></div></div></blockquote><div> </div></div></BODY></HTML>