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 696FA22899 for ; Tue, 10 Sep 2019 09:58:09 -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 6_Fx3xskdKMo for ; Tue, 10 Sep 2019 09:58:09 -0400 (EDT) Received: from smtp45.i.mail.ru (smtp45.i.mail.ru [94.100.177.105]) (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 2761C21696 for ; Tue, 10 Sep 2019 09:58:09 -0400 (EDT) Date: Tue, 10 Sep 2019 16:57:51 +0300 From: Alexander Turenko Subject: [tarantool-patches] Re: [PATCH 1/1] test: add tests from tarantool source tree Message-ID: <20190910135751.wy2wtyceufrhncxc@tkn_work_nb> 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: Igor Munkin Cc: tarantool-patches@freelists.org I found the following difference with the current master: | --- a/luajit-tap/fold_bug_LuaJIT_505.test.lua 2019-09-10 15:45:04.886639461 +0300 | +++ b/test/fold_bug_LuaJIT_505.test.lua 2019-09-10 15:44:22.221641210 +0300 | @@ -8,7 +8,8 @@ | -- Test file to demonstrate Lua fold machinery icorrect behavior, details: | -- https://github.com/LuaJIT/LuaJIT/issues/505 | | -jit.opt.start("hotloop=1") | +jit.opt.start(0, "hotloop=1") | +require('jit.dump').start("+bti", "505.trace") | for _ = 1, 20 do | local value = "abc" | local pos_c = string.find(value, "c", 1, true) >From what it appears? If it is necessary, then move it to its own commit or describe in the commit message. Please, base your branch on top of a last commit in a default branch (named 'tarantool'). Now it contains one extra commit ('luajit: fix string.find recording'). Verified in-source build, works ok. Tried out-of-source build: small and luajit-tap tests are not run (just not found). If we'll create just symlinks in 'test' repository directory and will remove creating symlinks from test/CMakeLists.txt, then luajit-tap would work good, but small tests would not be built (and so small tests will be found). It seems it is subject for a separate issue: don't sure about test-run or tarantool. Something was changed in test-run I guess; maybe here: https://github.com/tarantool/test-run/commit/b42093abc0e7fe895ff0ac7c0685d016c783b3a1 Don't sure whether it worth to block on this problem or file an issue and going on. Please, ask Kirill for that. Other then that everything look okay for me. WBR, Alexander Turenko. On Thu, Sep 05, 2019 at 05:20:07PM +0300, Igor Munkin wrote: > Introduced structure has the following benefits: > * Excess testing machinery currently 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 > > NB!: All added test chunks use tarantool tap module > --- > test/fold_bug_LuaJIT_505.test.lua | 21 +++ > test/gh.test.lua | 17 +++ > test/suite.ini | 5 + > test/table_chain_bug_LuaJIT_494.test.lua | 178 +++++++++++++++++++++++ > test/unsink_64_kptr.test.lua | 44 ++++++ > 5 files changed, 265 insertions(+) > create mode 100755 test/fold_bug_LuaJIT_505.test.lua > create mode 100755 test/gh.test.lua > create mode 100644 test/suite.ini > create mode 100755 test/table_chain_bug_LuaJIT_494.test.lua > create mode 100755 test/unsink_64_kptr.test.lua