<div dir="ltr">Hello, thanks for the review of new patch version!<br><br>I've fixed commit message (added `build` prefix) and rebased<br>from master.<br><br>I think commit, mentionied at patch message ("cmake: remove<br>dynamic-list linker option") is related to #5266 issue.<br>It's because symbols exports from required libs into tarantool binary<br>earlier were configured with two options:<br>by `--whole-archive` option;<br>and `--dynamic-list,${exports_file}` option, where exports_file was created<br>with use of mkexports script (symbols was grabbed by nm -D from shared<br>libraries).<br><br>Also, I've checkouted "cmake: remove dynamic-list linker option"<br>commit, built static tarantool and ran icu-date tests. In result<br>tests failed with `undefined symbol` errors.<br>After that, I checkouted previous commit, built tarantool again<br>and icu-date tests succeed</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">пт, 25 сент. 2020 г. в 00:10, Vladislav Shpilevoy <<a href="mailto:v.shpilevoy@tarantool.org">v.shpilevoy@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">Hi! Thanks for the patch!<br>
<br>
I would add 'build: ' prefix to the commit message.<br>
<br>
On 23.09.2020 13:02, HustonMmmavr wrote:<br>
> After patch 03790ac5510648d1d9648bb2281857a7992d0593 ("cmake: remove<br>
> dynamic-list linker option") symbols exports was changed (now we have<br>
> to export required symbols manually).<br>
<br>
Actually after some thinking I am not sure it is because of that commit.<br>
Symbols exports were always done manually. That commit changed which<br>
symbols were hidden. So essentially it extended the exported symbols<br>
set, not shrunk it.<br>
<br>
And that makes me wonder how could it lead to #5266?<br>
<br>
> Icu symbols, required by icu-date<br>
> rock (as ffi calls) are unused at linkage stage of tarantool binary<br>
> and thrown away from it so icu-date won't work (in case of tarantool<br>
> static build).<br>
<br>
They would have been thrown anyway. It is not related to the dynamic-list<br>
option removed in the mentioned commit.<br>
<br>
> This patch fixes this behaviour by adding symbols<br>
> required by icu-date rock to symbols export list.<br>
> <br>
> Close #5266<br>
</blockquote></div>