<div dir="ltr"><div dir="ltr">Thanks!<div><br></div><div>@ChangeLog<br></div><div><br></div><div><span style="color:rgb(0,0,0);font-family:HelveticaNeue;font-size:12px">- Fixed static-build on ubuntu (gh-5024)</span></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">вт, 9 июн. 2020 г. в 19:33, Alexander Turenko <<a href="mailto:alexander.turenko@tarantool.org">alexander.turenko@tarantool.org</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">LGTM.<br>
<br>
I didn't verify it manually, but the change itself looks reasonable and<br>
CI pass (except flaky tests and known Mac OS X problem on Travis-CI).<br>
<br>
Sergey, can you look into the proposed change too?<br>
<br>
Alexander (Tikhonov), we added a CI job to verify such things within<br>
[1]. Can you, please, look how it appears that the job didn't catch the<br>
problem?<br>
<br>
> Issue: <a href="https://github.com/tarantool/tarantool/issues/5024" rel="noreferrer" target="_blank">https://github.com/tarantool/tarantool/issues/5024</a><br>
> Branch: <a href="https://github.com/tarantool/tarantool/tree/HustonMmmavr/gh-5024-static-build-ubuntu" rel="noreferrer" target="_blank">https://github.com/tarantool/tarantool/tree/HustonMmmavr/gh-5024-static-build-ubuntu</a><br>
<br>
Please, answer with a changelog entry for future release notes. We track<br>
user-visible behaviour changes this way: bugfixes and features. When<br>
there is a fix for some build target or configuration, we mention it<br>
too.<br>
<br>
It is just free form description of changes for a user to be placed on<br>
the [releases page][2]. We highlight with @ChangeLog 'tag' to bring a<br>
mainterner attention.<br>
<br>
Since it is the regression, it would be good to mention, when the<br>
problem appears (`git describe bad_commit` output), in the future<br>
release notes.<br>
<br>
For future patches: add it to a cover letter or under `---` for a<br>
signleton patchset (but do not add to a commit message).<br>
<br>
[1]: <a href="https://github.com/tarantool/tarantool/issues/4551" rel="noreferrer" target="_blank">https://github.com/tarantool/tarantool/issues/4551</a><br>
[2]: <a href="https://github.com/tarantool/tarantool/releases" rel="noreferrer" target="_blank">https://github.com/tarantool/tarantool/releases</a><br>
<br>
WBR, Alexander Turenko.<br>
</blockquote></div></div>