<HTML><BODY><p>I'm not quiet happy of cleanup roll back, but anyway I did it for all places that were changed by it, in places where the code was deleted I've left it, as it doesn't change the history of the code.<br><br>           Alexander</p><p><br data-mce-bogus="1"></p><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">
        Пятница, 16 августа 2019, 5:32 +03:00 от Alexander Turenko <alexander.turenko@tarantool.org>:<br>
        <br>
        <div id="">






<div class="js-helper js-readmsg-msg">
        <style type="text/css"></style>
        <div>
                
                
            <div id="style_15659227441201229892_BODY">I looked briefly and it seems to be okay. But, please, remove cleanup<br>
changes from the dockerfile: they hide real changes and bloats the<br>
patch.<br>
<br>
I'll look deeper when you'll update the patch.<br>
<br>
WBR, Alexander Turenko.<br>
<br>
                                 > -RUN set -x \<br>
> -    && yum -y install \<br>
> +RUN set -x && \<br>
> +    yum -y install \<br>
      <br>
I meant this kind of changes.<br>
<br>
> -RUN yum -y install ncurses-static readline-static zlib-static pcre-static glibc-static<br>
> +    yum -y install \<br>
> +        ncurses-static \<br>
> +        readline-static \<br>
> +        zlib-static \<br>
> +        pcre-static \<br>
> +        glibc-static && \<br>
> +    yum -y install \<br>
> +        python-devel \<br>
> +        python-pip<br>
<br>
This.<br>
<br>
> +WORKDIR /<br>
>  <br>
>  RUN set -x && \<br>
> -    cd / && \<br>
>      curl -O -L <a href="https://www.openssl.org/source/openssl-1.1.0h.tar.gz" target="_blank">https://www.openssl.org/source/openssl-1.1.0h.tar.gz</a> && \<br>
>      tar -xvf openssl-1.1.0h.tar.gz && \<br>
>      cd openssl-1.1.0h && \<br>
<br>
And this.<br>
<br>
</div>
            
        
                
        </div>

        
</div>


</div>
</blockquote>
<br>
<br>-- <br>Alexander Tikhonov<br></BODY></HTML>