From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp60.i.mail.ru (smtp60.i.mail.ru [217.69.128.40]) (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 9264346970E for ; Fri, 24 Jan 2020 05:24:19 +0300 (MSK) Date: Fri, 24 Jan 2020 05:24:26 +0300 From: Alexander Turenko Message-ID: <20200124022426.w6vzzajjjblkasil@tkn_work_nb> References: <20200117134036.87011-1-sergepetrenko@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20200117134036.87011-1-sergepetrenko@tarantool.org> Subject: Re: [Tarantool-patches] [PATCH] Update decNumber library List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Serge Petrenko Cc: tarantool-patches@dev.tarantool.org LGTM. It also interesting why the warning did not cause fail of the build. We set -Werror for a Debug build, so I would expect fail. It seems we don't pass the flag due to some reason. It would be good to look why (I don't insist, however). CCed Kirill. Kirill, please, push it to master, 2.3 and 2.2, but before that push 2125b626faa79fa6c1ca4d770d8088dd2b8d0803 (the previous update of decNumber submodule, which fixes build in some Mac OS environments) to 2.2. I asked for this in [1]. This will allow to cherry-pick this commit cleanly downward. [1]: https://lists.tarantool.org/pipermail/tarantool-patches/2020-January/013759.html WBR, Alexander Turenko. On Fri, Jan 17, 2020 at 04:40:36PM +0300, Serge Petrenko wrote: > Update decNumber library to silence the build warning produced on too > long integer constant. > --- > No issue. > https://github.com/tarantool/tarantool/tree/sp/silence-decnumber-warning-full-ci > > third_party/decNumber | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/third_party/decNumber b/third_party/decNumber > index ec9e5be5a..4f318a5d5 160000 > --- a/third_party/decNumber > +++ b/third_party/decNumber > @@ -1 +1 @@ > -Subproject commit ec9e5be5a888c9a3de149fa94a2853476f720a88 > +Subproject commit 4f318a5d5a1fce1f16f2d6a4e23aa5661647c779 > -- > 2.21.0 (Apple Git-122) >