From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp14.mail.ru (smtp14.mail.ru [94.100.181.95]) (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 D35EB43D678 for ; Thu, 24 Oct 2019 02:11:39 +0300 (MSK) Date: Thu, 24 Oct 2019 02:11:29 +0300 From: Alexander Turenko Message-ID: <20191023231129.dqgjmyndg33e3res@tkn_work_nb> References: <20191008162000.66364-1-sergepetrenko@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20191008162000.66364-1-sergepetrenko@tarantool.org> Subject: Re: [Tarantool-patches] [PATCH] build: remove libyaml from rpm/deb dependencies List-Id: Tarantool development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Serge Petrenko Cc: tarantool-patches@freelists.org, tarantool-patches@dev.tarantool.org LGTM. Added docbot comment to the issue. Pushed to master and 2.2 (as 47b91e90f2a4e23e70a1a6735af3de203ffd59f4 itself) with minor changes for the commit message (see below). Kirill, maybe we'll made the same changes in build process for 2.1 and 1.10? This will reduce differences between build / runtime requirements for different tarantool versions and so maybe will be simpler for users and developers both. The patchset seems to be the following: * 53fbb5879f6861e7df980d8e1db44dbef93e4ede Bump libyaml version * cdf37876189fa005a350d6b69397348354bb2073 Extend range of printable unicode characters * 7f4029c706a11acd09e875bb0f0fc52d66e7810d Bump libyaml version * 47b91e90f2a4e23e70a1a6735af3de203ffd59f4 build: enable bundled libyaml for all systems. If we'll do, please, update https://github.com/tarantool/doc/issues/960 WBR, Alexander Turenko. On Tue, Oct 08, 2019 at 07:20:00PM +0300, Serge Petrenko wrote: > After we started using bundled version of libyaml by default > (see commit 47b91e90f2a4e23e70a1a6735af3de203ffd59f4), we can remove it > from building dependencies for RPM and DEB packets. Word 'packages' are usually used in this context, 'packets' are in network context. I changed it. Also reformatted the paragraph to avoid premature line breaks. > > Closes #4442 > --- > https://github.com/tarantool/tarantool/issues/4442 > https://github.com/tarantool/tarantool/tree/sp/gh-4442-rm-libyaml-deps > > debian/control | 3 +-- > rpm/tarantool.spec | 1 - > 2 files changed, 1 insertion(+), 3 deletions(-)