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 E5AB26EC5D; Tue, 6 Apr 2021 19:56:40 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org E5AB26EC5D DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1617728201; bh=s3CRl64qe2S5Kc3QddgVth0586awhSLGN5qUW1kBwQs=; 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=DrpMoExKqrtoZqEt6rv3CBZRcY5xodOA+ob/CTXc2GKgkJzeLwvIsv5UCs4tAc/Y2 6kmA4KkNeCT8Uy+ftzM2Z7F3ap0Nbc4xQ3NtBoFk/b8r8G1RuAMIBHGkDupLx6w15S F/TIo1dd0BfkT74N3qwrcEJso/eAFrSLkLmHaPhM= Received: from smtpng2.m.smailru.net (smtpng2.m.smailru.net [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 44CF36EC5D for ; Tue, 6 Apr 2021 19:56:39 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 44CF36EC5D Received: by smtpng2.m.smailru.net with esmtpa (envelope-from ) id 1lTp0I-0005vp-7J; Tue, 06 Apr 2021 19:56:38 +0300 Date: Tue, 6 Apr 2021 19:56:27 +0300 To: Sergey Kaplun Message-ID: <20210406165627.GE29703@tarantool.org> References: <20210331225823.GT29703@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Clacks-Overhead: GNU Terry Pratchett User-Agent: Mutt/1.10.1 (2018-07-13) X-7564579A: B8F34718100C35BD X-77F55803: 4F1203BC0FB41BD912A3E3D5D4B49FC1FB6C11AAC314D64EDF2811C24C4EA09700894C459B0CD1B93D6639E9854C04EF05FE04D60BFFAB921102AB7BB65CCEBC37AD88C62730AFDD X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE763424119D34F5CBFEA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F79006375F0BD5CF353A411D8638F802B75D45FF5571747095F342E8C7A0BC55FA0FE5FC864C7434B91C461292C16262D4F0D35C14D97B24B04E7E8B389733CBF5DBD5E913377AFFFEAFD269176DF2183F8FC7C0A29E2F051442AF778941B15DA834481FCF19DD082D7633A0EF3E4896CB9E6436389733CBF5DBD5E9D5E8D9A59859A8B6E5E764EB5D94DBD4CC7F00164DA146DA6F5DAA56C3B73B237318B6A418E8EAB86D1867E19FE14079C09775C1D3CA48CF3D321E7403792E342EB15956EA79C166A417C69337E82CC275ECD9A6C639B01B78DA827A17800CE7BEA0E1DB66A6FB7C731C566533BA786AA5CC5B56E945C8DA X-C1DE0DAB: 0D63561A33F958A5BEB777A4F2760F3A2237A9862B5A79BE72EB6FD1610E9A0AD59269BC5F550898D99A6476B3ADF6B47008B74DF8BB9EF7333BD3B22AA88B938A852937E12ACA7502E6951B79FF9A3F410CA545F18667F91A7EA1CDA0B5A7A0 X-C8649E89: 4E36BF7865823D7055A7F0CF078B5EC49A30900B95165D340297C696F996E3847716B4A0ADC56F466DEB2A3A6D2B4CC13091BF44532E931FB9BDB2175D25FE121D7E09C32AA3244C87F61EA8924988448C8E1B7A77EBB45DB4DF56057A86259F927AC6DF5659F194 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu530nj6fImhcD4MUrOEAnl0W826KZ9Q+tr5ycPtXkTV4k65bRjmOUUP8cvGozZ33TWg5HZplvhhXbhDGzqmQDTd6OAevLeAnq3Ra9uf7zvY2zzsIhlcp/Y7m53TZgf2aB4JOg4gkr2biojAX27rRizpLncDGNpgZo+HQ== X-Mailru-Sender: 689FA8AB762F73936BC43F508A063822E230BAE6B45E63FEF9678E0333704B51A7C8D0F45F857DBFE9F1EFEE2F478337FB559BB5D741EB964C8C2C849690F8E70A04DAD6CC59E33667EA787935ED9F1B X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH v2 luajit 03/30] test: adapt Lua 5.1 suite for out-of-source build 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" Sergey, On 01.04.21, Sergey Kaplun wrote: > Igor, > > Thanks for the review! > > On 01.04.21, Igor Munkin wrote: > > Sergey, > > > > Thanks for the patch! Honestly, I'm not fond of the solution you > > implemented, though it allows to make as little changes as possible. > > I propose to implement something similar to the solution you've made > > within lua-Harness series: > > * You need two functions: one for launching source files for > > additional checks and another one for running temporary Lua chunks. > > * You also need two functions for the same reason. > > > > Mind the fact is implemented via , so basically, > > there might be no so much changes made for . Thoughts? > > I prefer to use one function, and two wrappers around it. > For example, let it be `absolute_dofile()` and `cwd_dofile()`. Why do you need the second one? > > But honestly, it is not the single one problem of this suite (no test > isolation, global variables, huge excess output, hardcoded limit > values, etc). Yes, the whole suite is a single problem, but you've already fixed something. This patch can fix the base problem we've resolved while adopting lua-Harness. Why do you want to omit it here? > It looks like we need to refactor this suite later (plus adapt test > for LuaJIT extensions from Lua 5.2+). I propose to do all this work > via that refactoring, not now. Otherwise, it looks like a patch, which > will still need to be refactored later. It's not about refactoring, it's about the general approach. François proposed an elegant solution with <_dofile> that can be overloaded in luajit-test-init.lua (and we have to introduce one into LuaJIT then), so I see no reason to postpone such simple change. Of course this issue can be done within #5790 (or not, again, since it relates to another suite), so if you don't want to do this, let's close the eyes and leave such unsighty workaround. Consider you have my LGTM here if nobody else is bothered. I'll just glance the patch once more. > > Thoughts? > > > > > -- > > Best regards, > > IM > > -- > Best regards, > Sergey Kaplun -- Best regards, IM