From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtpng3.m.smailru.net (smtpng3.m.smailru.net [94.100.177.149]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 877F7469719 for ; Fri, 25 Sep 2020 00:10:36 +0300 (MSK) References: <20200923110251.33201-1-huston.mavr@gmail.com> From: Vladislav Shpilevoy Message-ID: Date: Thu, 24 Sep 2020 23:10:34 +0200 MIME-Version: 1.0 In-Reply-To: <20200923110251.33201-1-huston.mavr@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Tarantool-patches] [PATCH v2] Add missed icu symbols List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: HustonMmmavr , tarantool-patches@dev.tarantool.org, yaroslav.dynnikov@gmail.com, alexander.turenko@tarantool.org, tsafin@tarantool.org Hi! Thanks for the patch! I would add 'build: ' prefix to the commit message. On 23.09.2020 13:02, HustonMmmavr wrote: > After patch 03790ac5510648d1d9648bb2281857a7992d0593 ("cmake: remove > dynamic-list linker option") symbols exports was changed (now we have > to export required symbols manually). Actually after some thinking I am not sure it is because of that commit. Symbols exports were always done manually. That commit changed which symbols were hidden. So essentially it extended the exported symbols set, not shrunk it. And that makes me wonder how could it lead to #5266? > Icu symbols, required by icu-date > rock (as ffi calls) are unused at linkage stage of tarantool binary > and thrown away from it so icu-date won't work (in case of tarantool > static build). They would have been thrown anyway. It is not related to the dynamic-list option removed in the mentioned commit. > This patch fixes this behaviour by adding symbols > required by icu-date rock to symbols export list. > > Close #5266