

<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;">Вторник, 14 апреля 2020, 11:07 +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_15868516700286323910_BODY">Hi,<br><br>Thanks for the patch. See 2 comments inline.<br><br>On 17:53 Mon 13 Apr , Alexander V. Tikhonov wrote:<br>> 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><br>1. From patch it is unclear what host has an invalid SSL certificate.<br>Could you elaborate?</div></div></div></div></blockquote></div><div>The root cause of the issue is not that SSL certificate wrong, but local host may have wrong local setup which produce the issue with SSL certificate checking, like date my be not synchronized either some other issues, which we don’t want to fix. That is why the regular <a href="https://files.pythonhosted.org">https://files.pythonhosted.org</a> host may have SSL certificate which will block the installation process even it is fine, but the target host may not approve it. So the better way to avoid our hosts from it is to block just for this formula SSL certificate even we know the host name and prove it.</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><br>> Close #4883<br>> ---<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>> Issue: <a href="https://github.com/tarantool/tarantool/issues/4883" target="_blank">https://github.com/tarantool/tarantool/issues/4883</a><br>><br>> .travis.mk | 23 +++++++++++++++++------<br>> 1 file changed, 17 insertions(+), 6 deletions(-)<br>><br>> diff --git a/.travis.mk b/.travis.mk<br>> index 001752844..68cc12c0b 100644<br>> --- a/.travis.mk<br>> +++ b/.travis.mk<br>> @@ -154,22 +154,33 @@ test_static_docker_build:<br>> # OSX #<br>> #######<br>><br>> -# since Python 2 is EOL it's latest commit from tapped local formula is used<br>> OSX_PKGS=openssl readline curl icu4c libiconv zlib autoconf automake libtool \<br>> - cmake file://${PWD}/tools/brew_taps/tntpython2.rb<br>> + cmake<br>><br>> deps_osx:<br>> - # install brew using command from Homebrew repository instructions:<br>> + # Install brew using command from Homebrew repository instructions:<br>> # <a href="https://github.com/Homebrew/install" target="_blank">https://github.com/Homebrew/install</a><br>> # NOTE: 'echo' command below is required since brew installation<br>> # script obliges the one to enter a newline for confirming the<br>> # installation via Ruby script.<br>> brew update || echo | /usr/bin/ruby -e \<br>> "$(curl -fsSL <a href="https://raw.githubusercontent.com/Homebrew/install/master/install" target="_blank">https://raw.githubusercontent.com/Homebrew/install/master/install</a>)"<br>> - # try to install the packages either upgrade it to avoid of fails<br>> - # if the package already exists with the previous version<br>> + # Try to install the packages either upgrade it to avoid of fails<br>> + # if the package already exists with the previous version.<br>> brew install --force ${OSX_PKGS} || brew upgrade ${OSX_PKGS}<br>> - pip install --force-reinstall -r test-run/requirements.txt<br>> + # Since Python 2 is EOL, it's latest commit from tapped local formula is<br>> + # used. Some packages from tntpython2.rb formula use external download<br>> + # hosts which do not have valid SSL certificate. To resolve it the SSL<br>> + # certificates check need to be disabled during formula installation.<br>> + echo insecure >>${HOME}/.curlrc<br>> + brew install --force file://${PWD}/tools/brew_taps/tntpython2.rb || :<br>> + sed '$d' ${HOME}/.curlrc >${HOME}/.curlrc.new && \<br>> + mv ${HOME}/.curlrc.new ${HOME}/.curlrc<br><br>2. Failed to understand this magick with .curlrc. Could you explain?</div></div></div></div></blockquote></div><div>Actually seems that I forgot to optimize the temporary code - fixed it to «sed … -i .curlrc». By the way sed here removes last line from the file.</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><br>> + python2 -V<br>> + pip install --trusted-host files.pythonhosted.org \<br>> + --upgrade pip setuptools<br>> + pip install --trusted-host files.pythonhosted.org \<br>> + --force-reinstall -r test-run/requirements.txt<br>><br>> build_osx:<br>> cmake . -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_WERROR=ON ${CMAKE_EXTRA_PARAMS}<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>