

<HTML><BODY><div>Hi Sergey, thanks for the review, please check my comments below.<br><br> <blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Пятница, 10 апреля 2020, 17:34 +03:00 от Sergey Bronnikov <sergeyb@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><style type="text/css"></style><div><div id="style_15865292901608118097_BODY">Hi,<br><br>Sasha, thanks for the patch. See my comment below.<br><br>On 11:40 Thu 09 Apr , Alexander V. Tikhonov wrote:<br>> gitlab-ci: add Catalina OSX 10.15<br>><br>> Added Catalina OSX 10.15 to gitlab-ci testing and removed OSX 10.13,<br>> due to decided to have only 2 last major releases, for now it is<br>> 10.14 and 10.15 OSX versions. Also changed the commit job for branches<br>> from 10.14 to 10.15 OSX version.<br>><br>> Added to OSX 10.15 additional call to /etc/profile to setup the PATH,<br>> otherwise it doesn't have the path to brew installed packages.<br><br>PATH can be adjusted in runtime with 'export' call and by using<br>user-specific $HOME/.profile and reloading it to current environemnt.<br>What are the reasons to touch system-wide file?</div></div></div></div></blockquote></div><div>Unfortunately we can’t be sure what path is really needed, that is why I used to call</div><div>/etc/profile file for PATH setup, by the internal call to special tool ‘/usr/libexec/path_helper’<br>which can detect needed paths itself. Anyway, may be this way is not clear and better</div><div>to use here the script that is needed, that is why I’ve changed the call of /etc/profile to</div><div>/usr/libexec/path_helper.</div><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>> Formula tntpython2.rb consist of the packages which download target host<br>> does not have valid SSL certificate, disabled curl SSL check for it,<br>> also the same issue resolved for pip packages.<br>><br>> Github: <a href="https://github.com/tarantool/tarantool/tree/avtikhon/osx_15_catalina-full-ci" target="_blank">https://github.com/tarantool/tarantool/tree/avtikhon/osx_15_catalina-full-ci</a><br>><br>> Alexander V. Tikhonov (2):<br>> build: disable at OSX curl SSL<br>> gitlab-ci: add Catalina OSX 10.15<br>><br>> .gitlab-ci.yml | 11 ++++++-----<br>> .travis.mk | 15 +++++++++++++--<br>> 2 files changed, 19 insertions(+), 7 deletions(-)<br>><br>> --<br>> 2.17.1<br>><br><br>--<br>sergeyb@</div></div></div></div></blockquote> <div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Alexander Tikhonov</div></div></div><div> </div></div></BODY></HTML>