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 A2FA4376364; Wed, 15 Mar 2023 19:15:18 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org A2FA4376364 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1678896918; bh=1UM9XJ8oL5wdxDN/j8+rrHOi19cQ4VR6SGNb+WhKaEU=; h=To:Date:In-Reply-To:References:Subject:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=je0ZRQT1T8hDdtplIDsqpE6FjaIr9tgc+79/x7LQtIAvGBpk1DSSpex6DHF2gsEnj z9SOqhZW2Td7OIfR/VHknwR/+LHSt+bc3QsfyruZ2imtSCXpeqG48fnwrT/iiKPiAl EgeiaxeGLQQqQcIaskxxJJ7NWsubU/z1sOI+/alc= Received: from smtpng1.i.mail.ru (smtpng1.i.mail.ru [94.100.181.251]) (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 6B7E8376364 for ; Wed, 15 Mar 2023 19:14:51 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 6B7E8376364 Received: by smtpng1.m.smailru.net with esmtpa (envelope-from ) id 1pcTm6-0003ZV-KF; Wed, 15 Mar 2023 19:14:51 +0300 To: Igor Munkin , Maxim Kokryashkin Date: Wed, 15 Mar 2023 19:11:01 +0300 Message-Id: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Mailru-Src: smtp X-4EC0790: 10 X-7564579A: EEAE043A70213CC8 X-77F55803: 4F1203BC0FB41BD91FB0073BE751D55461ABC3E74F83D87504755FF90D1007B7182A05F5380850404C228DA9ACA6FE2766C27E5A0CFE3F43F62631E9B17C2E7F83EBE3B8FC97A507F1CF934CAA1CF663 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7B7733D0215A2F71AEA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F790063765D69633A07E2F0F8638F802B75D45FF36EB9D2243A4F8B5A6FCA7DBDB1FC311F39EFFDF887939037866D6147AF826D825961B71BBE6B1DE77F3E12B42900B49117882F4460429724CE54428C33FAD305F5C1EE8F4F765FCA87D11164F1816E7A471835C12D1D9774AD6D5ED66289B52BA9C0B312567BB23117882F4460429728776938767073520C65AC60A1F0286FEF6B57BC7E6449061A352F6E88A58FB86F5D81C698A659EA7E827F84554CEF5019E625A9149C048EE9ECD01F8117BC8BEE2021AF6380DFAD18AA50765F790063735872C767BF85DA227C277FBC8AE2E8B9F5955FECEF5819E75ECD9A6C639B01B4E70A05D1297E1BBCB5012B2E24CD356 X-C8649E89: 4E36BF7865823D7055A7F0CF078B5EC49A30900B95165D346E67BC984B8ABCB56D64DEECF87BD6119224B606CD4DDC0132F6AC50CC0298CC54D23EFCDCA77B5F1D7E09C32AA3244C0E6C48F9B090292875F3D97B46FBA45335DA7DC5AF9B58C0927AC6DF5659F194 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu530nj6fImhcD4MUrOEAnl0W826KZ9Q+tr5ycPtXkTV4k65bRjmOUUP8cvGozZ33TWg5HZplvhhXbhDGzqmQDTd6OAevLeAnq3Ra9uf7zvY2zzsIhlcp/Y7m53TZgf2aB4JOg4gkr2biojOXwBDQf4j7M9ulE1WNdRCQ== X-DA7885C5: 9AAAE34ECA47252F992EB771DD7126D07A8602D8D343BAB47675A8283C4A3902262E2D401490A4A0DB037EFA58388B346E8BC1A9835FDE71 X-Mailru-Sender: 689FA8AB762F73933AF1F914F131DBF53BD8B5E69037F9BCE7A724FE2702A0110FBE9A32752B8C9C2AA642CC12EC09F1FB559BB5D741EB962F61BD320559CF1EFD657A8799238ED55FEEDEB644C299C0ED14614B50AE0675 X-Mras: Ok Subject: [Tarantool-patches] [PATCH v1 luajit 1/5] test: fix setting of {DY}LD_LIBRARY_PATH variables 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: Sergey Kaplun via Tarantool-patches Reply-To: Sergey Kaplun Cc: tarantool-patches@dev.tarantool.org Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" When we set `LUA_TEST_ENV_MORE` variable to be used in the additional env command for run testing if `"` is used to wrap `LD_LIBRARY_PATH` value the content of this environment variable is literally `"/abs/path1:/abs/path2:...:". So, the first entry is treated as the relative path starting with `"`. In that case if we need to library to be loaded via FFI for this particular test, that loading fails with the error "cannot open shared object file", since the path to it is incorrect. This patch removes `"` wrapping for the aforementioned variables. --- You can test the behaviour of the patch with the following diff: | diff --git a/test/tarantool-tests/CMakeLists.txt b/test/tarantool-tests/CMakeLists.txt | index a428d009..86dc916a 100644 | --- a/test/tarantool-tests/CMakeLists.txt | +++ b/test/tarantool-tests/CMakeLists.txt | @@ -65,9 +65,9 @@ add_subdirectory(lj-49-bad-lightuserdata) | add_subdirectory(lj-416-xor-before-jcc) | add_subdirectory(lj-601-fix-gc-finderrfunc) | add_subdirectory(lj-727-lightuserdata-itern) | -add_subdirectory(lj-flush-on-trace) | add_subdirectory(misclib-getmetrics-capi) | add_subdirectory(misclib-sysprof-capi) | +add_subdirectory(lj-flush-on-trace) | | # The part of the memory profiler toolchain is located in tools | # directory, jit, profiler, and bytecode toolchains are located test/tarantool-tests/CMakeLists.txt | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/test/tarantool-tests/CMakeLists.txt b/test/tarantool-tests/CMakeLists.txt index a428d009..38d6ae49 100644 --- a/test/tarantool-tests/CMakeLists.txt +++ b/test/tarantool-tests/CMakeLists.txt @@ -102,6 +102,11 @@ endif() # loaded modules on MacOS instead of shared libraries as it is # done on Linux and BSD, another environment variable should be # used to guide while searching the extension. +# XXX: Be noticed that we shouldn't use `"` here to wrap +# the variable's content. If we do this, the variable value will +# contain `"` at the beginning and the end, so this `"` at the +# beginning will be treated as the directory for the first entry +# (the last subdirectory added). if(CMAKE_SYSTEM_NAME STREQUAL "Darwin") # XXX: Apple tries their best to "protect their users from # malware". As a result SIP (see the link[1] below) has been @@ -122,9 +127,9 @@ if(CMAKE_SYSTEM_NAME STREQUAL "Darwin") # # [1]: https://support.apple.com/en-us/HT204899 # [2]: https://developer.apple.com/library/archive/documentation/Security/Conceptual/System_Integrity_Protection_Guide/RuntimeProtections/RuntimeProtections.html - list(APPEND LUA_TEST_ENV_MORE DYLD_LIBRARY_PATH="${LD_LIBRARY_PATH}") + list(APPEND LUA_TEST_ENV_MORE DYLD_LIBRARY_PATH=${LD_LIBRARY_PATH}) else() - list(APPEND LUA_TEST_ENV_MORE LD_LIBRARY_PATH="${LD_LIBRARY_PATH}") + list(APPEND LUA_TEST_ENV_MORE LD_LIBRARY_PATH=${LD_LIBRARY_PATH}) endif() # LUA_CPATH and LD_LIBRARY_PATH variables and also TESTLIBS list -- 2.34.1