

<HTML><BODY><div>Hi!</div><div>Thanks for the patch!</div><div>LGTM, except for the commit message.</div><div> </div><div> </div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Среда, 15 марта 2023, 19:14 +03:00 от Sergey Kaplun <skaplun@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16788968911607925128_BODY">When we set `LUA_TEST_ENV_MORE` variable to be used in the additional<br>env command for run testing if `"` is used to wrap `LD_LIBRARY_PATH`</div></div></div></div></blockquote><div>Typo: s/wrap/wrap the</div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div><div class="js-helper js-readmsg-msg"><div><div>value the content of this environment variable is literally<br>`"/abs/path1:/abs/path2:...:". So, the first entry is treated as the</div></div></div></div></blockquote><div>Typo: I guess you have forgot a ` here.</div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div><div class="js-helper js-readmsg-msg"><div><div>relative path starting with `"`. In that case if we need to library to</div></div></div></div></blockquote><div>Typo: s/to library/the library</div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div><div class="js-helper js-readmsg-msg"><div><div>be loaded via FFI for this particular test, that loading fails with the<br>error "cannot open shared object file", since the path to it is<br>incorrect.<br><br>This patch removes `"` wrapping for the aforementioned variables.<br>---</div></div></div></div></blockquote><div><div>--<br>Best regards,</div><div>Maxim Kokryashkin</div></div><div> </div></BODY></HTML>