From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from [87.239.111.99] (localhost [127.0.0.1]) by dev.tarantool.org (Postfix) with ESMTP id 129556EC40; Tue, 6 Jul 2021 23:52:41 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 129556EC40 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1625604761; bh=D7PX+pT8uyU0DalAnRw4yEhHzUIZt89RUikkfyurEBc=; h=Date:To:References:In-Reply-To:Subject:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=AWWCTsGQwcHQH/5bDHZq4fi3XGpPxWZ9gJOqZe8DQrXT+frW81DIiQTR/HIJGB/Mu 5xGKSgbg8YL0GushE1tK3POQZgSwAt0o3MLhpYDHWZdGDYBZ4ZrLApg1GCyeKzqRoL XWOn5H2My+o+C8YlM+FhnDcEeY6utgQxFT1VhHXs= Received: from smtpng2.i.mail.ru (smtpng2.i.mail.ru [94.100.179.3]) (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 3BF2E6EC40 for ; Tue, 6 Jul 2021 23:52:40 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 3BF2E6EC40 Received: by smtpng2.m.smailru.net with esmtpa (envelope-from ) id 1m0s3b-0004x5-0t; Tue, 06 Jul 2021 23:52:39 +0300 Date: Tue, 6 Jul 2021 23:29:08 +0300 To: Maxim Kokryashkin Message-ID: <20210706202908.GA11494@tarantool.org> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-Clacks-Overhead: GNU Terry Pratchett User-Agent: Mutt/1.10.1 (2018-07-13) X-4EC0790: 10 X-7564579A: 646B95376F6C166E X-77F55803: 4F1203BC0FB41BD954DFF1DC42D673FB2F1AA0EB8A504C8721532AB396CDCF09182A05F53808504052043027AD0E4CCF5FB58AC0AD6F5BCBCF4A549380B009CBA081B1514F410D45 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE79145AB6E9E75F07EEA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F79006370F98874192B1BA168638F802B75D45FF36EB9D2243A4F8B5A6FCA7DBDB1FC311F39EFFDF887939037866D6147AF826D8214D9AD0F8F6EB3269EE66709626C529117882F4460429724CE54428C33FAD305F5C1EE8F4F765FC292D688DDAD4E7BC389733CBF5DBD5E9C8A9BA7A39EFB766F5D81C698A659EA7CC7F00164DA146DA9985D098DBDEAEC80A9EC8C3488E7643F6B57BC7E6449061A352F6E88A58FB86F5D81C698A659EA7E827F84554CEF5019E625A9149C048EE9ECD01F8117BC8BEE2021AF6380DFAD18AA50765F790063735872C767BF85DA227C277FBC8AE2E8B569F1129A2C6445075ECD9A6C639B01B4E70A05D1297E1BBCB5012B2E24CD356 X-C1DE0DAB: 0D63561A33F958A541E4A5C6E1E33A927FC39433B7EF4305AD639258B582DF9FD59269BC5F550898D99A6476B3ADF6B47008B74DF8BB9EF7333BD3B22AA88B938A852937E12ACA75342909995EBBA6E4410CA545F18667F91A7EA1CDA0B5A7A0 X-C8649E89: 4E36BF7865823D7055A7F0CF078B5EC49A30900B95165D3458239C6F30CE7ACFC427855B57F09BD8ECE93FEBBD66D907BD1BC032E26A28BCE662DCC8A59C33DD1D7E09C32AA3244C3D091E5030282773F22758ABAE5ACE03BBA718C7E6A9E042927AC6DF5659F194 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu530nj6fImhcD4MUrOEAnl0W826KZ9Q+tr5ycPtXkTV4k65bRjmOUUP8cvGozZ33TWg5HZplvhhXbhDGzqmQDTd6OAevLeAnq3Ra9uf7zvY2zzsIhlcp/Y7m53TZgf2aB4JOg4gkr2biojwWhFrYo6Pn2Vo+W/n/dOaw== X-Mailru-Sender: 689FA8AB762F7393C37E3C1AEC41BA5D63409BF72378345BF6B4DF930BF4EC98A7C8D0F45F857DBFE9F1EFEE2F478337FB559BB5D741EB964C8C2C849690F8E70A04DAD6CC59E33667EA787935ED9F1B X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH 0/4] test: bump lua-Harness suite X-BeenThere: tarantool-patches@dev.tarantool.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Igor Munkin via Tarantool-patches Reply-To: Igor Munkin Cc: tarantool-patches@dev.tarantool.org Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" Max, Thanks for the series! I see the issue is described not quite clear and you are misguided a bit, so I decided to describe how I see this patchset to be done. 1. As you can see here[1], we borrowed only the part of lua-Harness repo at the particular revision. Hence we need to fetch only the changes related to test_lua directory (but also consider the changes around it). 2. Right after the commit a74be27 on the master branch, you can find commit (ac57a6c), that relates to the sources located outside of the desired directory. Just skip it and the other similar changes. 3. Then you can see commit (074fefa), that changes the default assertion machinery used within lua-Harness suite. This patch has to be backported. 4. Here we come to 4 commits made due to my complains to Francois. They are already in the series and I left several comments per patch. 5. Again, another single commit (9455281) with changes in test_lua, but which are irrelevant to Tarantool. Anyway, should be backported. 6. Finally, the patch introducing Tarantool profile for lua-Harness (d3ceee1). Furthermore, in scope of this commit CLI tests are disabled the right way. Since this is relevant for Tarantool infrastructure, this should be backported to our trunk as a separate commit. 7. The next commit (24a570c) fixes the issue found by our CI, but Francois has committed the fix also working on Windows (since Microsloth can't use normal slashes...). Also should be backported. 8. The last commit to be backported separately is the next one (1da5b1b). It enables CLI tests disabled earlier in (6). Should be backported too. 9. Everything else on the master branch (HEAD is 673c786 at the moment of writing) can be backported within a single commit. Actually, I'm for moving the patch in (5) in scope of this commit. Considering everything above, I see the following history: | $ git log --oneline | XXXXXXX test: update lua-Harness to 673c786 | 1da5b1b test: support tarantool cli in lua-Harness | 24a570c test: backport lua-Harness directory detection | d3ceee1 test: support tarantool in lua-Harness | 60da289 test: refactor with _dofile | 1be25a8 test: refactor with _retrieve_progname | 6c2aa87 test: use CI friendly variables in lua-Harness | 8041c45 test: rename lua-Harness tap to test_assertion | 074fefa test: port lua-Harness to Test.Assertion As for 'XXXXXXX' patch, it contains of the following: * 673c7869 https * 97e9e4c1 check if luac exists * 4db7e539 fix when no debug * ac7671b6 follow LuaJIT * 70404580 refactor with near * 9455281b follow ravi 1.0-beta4 After everything above is done, we will be synced with Francois' repo and also will have a nice history of the changes related to our LuaJIT fork and Tarantool. On 05.07.21, Maxim Kokryashkin wrote: > From: Maxim Kokryashkin Please, use this name and email for the commits. Now everything is authored with the mail you are using to send the patches. > > As our experience has been considered by the maintainer of lua-Harness > and the issues we faced are finally fixed in mainline repo, we should > bump lua-Harness suite up to 7040458. > > Part of tarantool/tarantool#5970 I try to make the mess with tag usage a bit clear to you below: | $ git log --oneline | XXXXXXX test: update lua-Harness to 673c786 # Resolves #5970 and Part of #4473 | 1da5b1b test: support tarantool cli in lua-Harness # Part of #5970 and Part of #4473 | 24a570c test: backport lua-Harness directory detection # Part of #5970 and Part of #4473 | d3ceee1 test: support tarantool in lua-Harness # Part of #5970 and Part of #4473 | 60da289 test: refactor with _dofile # Part of #5970 and Part of #4473 | 1be25a8 test: refactor with _retrieve_progname # Part of #5970 and Part of #4473 | 6c2aa87 test: use CI friendly variables in lua-Harness # Part of #5970 and Part of #4473 | 8041c45 test: rename lua-Harness tap to test_assertion # Part of #5970 and Part of #4473 | 074fefa test: port lua-Harness to Test.Assertion # Part of #5970 and Part of #4473 > > Maxim Kokryashkin (4): > test: resolving program name > test: out-of-source testing > test: CI-environment > test: TAP module name collisions > > test/lua-Harness-tests/090-tap.t | 2 +- > test/lua-Harness-tests/091-profile.t | 2 +- > test/lua-Harness-tests/101-boolean.t | 4 ++-- > test/lua-Harness-tests/102-function.t | 4 ++-- > test/lua-Harness-tests/103-nil.t | 4 ++-- > test/lua-Harness-tests/104-number.t | 4 ++-- > test/lua-Harness-tests/105-string.t | 4 ++-- > test/lua-Harness-tests/106-table.t | 4 ++-- > test/lua-Harness-tests/107-thread.t | 4 ++-- > test/lua-Harness-tests/108-userdata.t | 4 ++-- > test/lua-Harness-tests/200-examples.t | 2 +- > test/lua-Harness-tests/201-assign.t | 2 +- > test/lua-Harness-tests/202-expr.t | 2 +- > test/lua-Harness-tests/203-lexico.t | 10 +++++----- > test/lua-Harness-tests/204-grammar.t | 2 +- > test/lua-Harness-tests/211-scope.t | 2 +- > test/lua-Harness-tests/212-function.t | 2 +- > test/lua-Harness-tests/213-closure.t | 2 +- > test/lua-Harness-tests/214-coroutine.t | 2 +- > test/lua-Harness-tests/221-table.t | 2 +- > test/lua-Harness-tests/222-constructor.t | 2 +- > test/lua-Harness-tests/223-iterator.t | 2 +- > test/lua-Harness-tests/231-metatable.t | 4 ++-- > test/lua-Harness-tests/232-object.t | 2 +- > test/lua-Harness-tests/241-standalone.t.disabled | 4 ++-- > test/lua-Harness-tests/242-luac.t | 4 ++-- > test/lua-Harness-tests/301-basic.t | 6 +++--- > test/lua-Harness-tests/303-package.t | 6 +++--- > test/lua-Harness-tests/304-string.t | 2 +- > test/lua-Harness-tests/305-utf8.t | 4 ++-- > test/lua-Harness-tests/306-table.t | 2 +- > test/lua-Harness-tests/307-math.t | 2 +- > test/lua-Harness-tests/308-io.t | 4 ++-- > test/lua-Harness-tests/309-os.t | 8 ++++---- > test/lua-Harness-tests/310-debug.t | 2 +- > test/lua-Harness-tests/311-bit32.t | 2 +- > test/lua-Harness-tests/314-regex.t | 2 +- > test/lua-Harness-tests/320-stdin.t | 4 ++-- > test/lua-Harness-tests/401-bitop.t | 2 +- > test/lua-Harness-tests/402-ffi.t | 2 +- > test/lua-Harness-tests/403-jit.t | 2 +- > test/lua-Harness-tests/404-ext.t | 4 ++-- > test/lua-Harness-tests/411-luajit.t.disabled | 4 ++-- > test/lua-Harness-tests/CMakeLists.txt | 5 ----- > .../{tap.lua => test_assertion.lua} | 12 ++++++++++++ > 45 files changed, 82 insertions(+), 75 deletions(-) > rename test/lua-Harness-tests/{tap.lua => test_assertion.lua} (95%) > > -- > 2.31.1 > [1]: https://github.com/tarantool/luajit/commit/347b8bf -- Best regards, IM