From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id 6994A2310C for ; Wed, 11 Sep 2019 14:22:23 -0400 (EDT) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id EwHFJBP4aO9L for ; Wed, 11 Sep 2019 14:22:23 -0400 (EDT) Received: from smtp37.i.mail.ru (smtp37.i.mail.ru [94.100.177.97]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id B812123108 for ; Wed, 11 Sep 2019 14:22:22 -0400 (EDT) Date: Wed, 11 Sep 2019 21:22:17 +0300 From: Kirill Yukhin Subject: [tarantool-patches] Re: [PATCH 0/1] Move luajit-related tests to luajit repo Message-ID: <20190911182217.mfqojalk7g72cn4p@tarantool.org> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-Help: List-Unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-Subscribe: List-Owner: List-post: List-Archive: To: tarantool-patches@freelists.org Cc: Igor Munkin Hello, On 05 Sep 17:01, Igor Munkin wrote: > As discussed offline with Sasha Turenko I propose to move > luajit-related tests to its repo and adjust testing procedure to > search these chunks in luajit submodule directory. > > The most valuable benefits of this approach are the following: > > * Excess testing machinery is not required directly in luajit repo > considering whole CI process setup for tarantool and its static > linking with libluajit > * All new tests related to luajit fixes and enhancements can be added > within a single patch with the corresponding changeset > > Branch: https://github.com/tarantool/tarantool/tree/imun/gh-4478-move-lj-tests > Issue: https://github.com/tarantool/tarantool/issues/4478 > > Igor Munkin (1): > test: move luajit-tap suite to luajit repo I've checked your patch into 1.10, 2.1, 2.2 and master. -- Regards, Kirill Yukhin