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 9F5CC6ECDE; Tue, 18 Aug 2026 13:47:55 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 9F5CC6ECDE DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1787050075; bh=0WcpPBlcvRFj+EkVJTeyFUQkK09naOsLzY4b280C5YA=; 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=jaAvBs8Qjqr9Tmd55bSWdiP4vBms7/ZAXsqPxfHe2UuYmsJwAY0UIiA6NMWE8IVSU 1aF7tsVjWx3eOGkb3QWNoYqVDoGnqHFd9YLZmOtdDS1CW0eULi4yhY0QUbVMPdDjm9 C01qN00LFHG3ofPdL4bA+V2ZIbqJbhcQyza7T/68= Received: from send151.i.mail.ru (send151.i.mail.ru [89.221.237.246]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 26E396ECDE for ; Tue, 18 Aug 2026 13:47:54 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 26E396ECDE Received: by exim-smtp-67896c5f6-6t2dc with esmtpa (envelope-from ) id 1wwHMH-00000000Bv3-0GK9; Tue, 18 Aug 2026 13:47:53 +0300 Date: Tue, 18 Aug 2026 13:47:24 +0300 To: Evgeniy Temirgaleev Message-ID: References: <9d4e2c67dcc3640a917a649cfcd2f19253ddd2f1.1786019520.git.e.temirgaleev@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <9d4e2c67dcc3640a917a649cfcd2f19253ddd2f1.1786019520.git.e.temirgaleev@tarantool.org> X-Mailru-Src: smtp X-4EC0790: 10 X-7564579A: B8F34718100C35BD X-77F55803: 4F1203BC0FB41BD9E3E322B45F7E28238FB89EB72E4026CC08A64351E58551E1182A05F5380850404B6D60BFEA53A72C3DE06ABAFEAF6705474D5A6E1AD3099849ACE272206F778E9A2ED5FB3C81BEA3 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7B9115C1E69829868EA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F7900637F88016AB904663428638F802B75D45FF914D58D5BE9E6BC1A93B80C6DEB9DEE97C6FB206A91F05B29743B0B8F5A928532E070BE324C7D3C46F1E41163EBCB8E2F6B57BC7E64490618DEB871D839B73339E8FC8737B5C224936DA1BED736F9328CC7F00164DA146DAFE8445B8C89999729449624AB7ADAF37F6B57BC7E64490611E7FA7ABCAF51C92176DF2183F8FC7C078FCF50C7EAF9C588941B15DA834481F9449624AB7ADAF37BA3038C0950A5D3613377AFFFEAFD269176DF2183F8FC7C070C8F79A57DAFDD17B076A6E789B0E97A8DF7F3B2552694AD5FFEEA1DED7F25D49FD398EE364050FB28585415E75ADA9CEDA8D6C8C3B0531B3661434B16C20ACC84D3B47A649675FE827F84554CEF5019E625A9149C048EE9ECD01F8117BC8BEE2021AF6380DFAD18AA50765F790063735872C767BF85DA227C277FBC8AE2E8BC6A536F79815AD9275ECD9A6C639B01B4E70A05D1297E1BBCB5012B2E24CD356 X-C1DE0DAB: 0D63561A33F958A52A2C71BFBE7688E75002B1117B3ED696CAB07EABA09FE325559C6C5561145D6F823CB91A9FED034534781492E4B8EEAD9CFA8CFAC159CE19BDAD6C7F3747799A X-C8649E89: 1C3962B70DF3F0AD73CAD6646DEDE191716CD42B3DD1D34CAB70F9BE574AE9C625B6776AC983F447FC0B9F89525902EE6F57B2FD27647F25E66C117BDB76D659162A7FD5F9A2FA9616409F7BD4D8448C240CF5A3C08B4E80AC088C36372FD075E7A4AD82B81528B5B8341EE9D5BE9A0A6D67B559345786690225AEF4E7922E3F02162658268AE9376536EB022892E5344C41F94D744909CECFA6C6B0C050A61A8CAF69B82BA93681CD72808BE417F3B9E0E7457915DAA85F X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu53w8ahmwBjZKM/YPHZyZHvz5uv+WouB9+ObcCpyrx6l7KImUglyhkEat/+ysWwi0gdhEs0JGjl6ggRWTy1haxBpVdbIX1nthFXMZebaIdHP2ghjoIc/363UZI6Kf1ptIMVd9Ysz+d4RqWwLtVZxhi1w4= X-Mailru-Sender: 583F1D7ACE8F49BD1E22B6041E04C8AA9E57B523C78869A7F7A6B01B8567BB201CDAB0DE5D6A1FBD6D46CB22DF8743B0F2400F607609286E924004A7DEC283833C7120B22964430C52B393F8C72A41A84198E0F3ECE9B5443453F38A29522196 X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH luajit v3 2/3] ci: enabled sanitizer tests for macOS 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" Hi, Evgeniy! Thanks for the patch and fixes! Generally LGTM, but please clarify my questions below. On 06.08.26, Evgeniy Temirgaleev wrote: > From: Temir Galeev > > The arm64 and x86_64 architectures with clang/gcc compiler were added > to the matrix. > --- > .../README.md | 0 > .../action.yml | 8 +- > .../actions/setup-sanitizers-macos/README.md | 15 +++ > .../actions/setup-sanitizers-macos/action.yml | 73 +++++++++++++ > .github/workflows/sanitizers-testing.yml | 101 ++++++++++++++++-- > 5 files changed, 186 insertions(+), 11 deletions(-) > rename .github/actions/{setup-sanitizers => setup-sanitizers-linux}/README.md (100%) > rename .github/actions/{setup-sanitizers => setup-sanitizers-linux}/action.yml (76%) > create mode 100644 .github/actions/setup-sanitizers-macos/README.md > create mode 100644 .github/actions/setup-sanitizers-macos/action.yml > > diff --git a/.github/actions/setup-sanitizers/README.md b/.github/actions/setup-sanitizers-linux/README.md > similarity index 100% > rename from .github/actions/setup-sanitizers/README.md > rename to .github/actions/setup-sanitizers-linux/README.md > diff --git a/.github/actions/setup-sanitizers/action.yml b/.github/actions/setup-sanitizers-linux/action.yml > similarity index 76% > rename from .github/actions/setup-sanitizers/action.yml > rename to .github/actions/setup-sanitizers-linux/action.yml > index 8642d553..18f5a75d 100644 > --- a/.github/actions/setup-sanitizers/action.yml > +++ b/.github/actions/setup-sanitizers-linux/action.yml > @@ -20,13 +20,17 @@ runs: > - name: Install build and test dependencies > run: | > apt -y update > + echo Available compilers: > + export CC_FAMILY=`echo ${CC_NAME} | sed 's/-.*$//'` > + apt list | grep -Pe "^${CC_FAMILY}-[0-9]+/" > + # Try to install > apt -y install ${CC_NAME} libstdc++-10-dev cmake ninja-build make perl > shell: bash > env: > CC_NAME: ${{ inputs.cc_name }} > - - name: Set specific C compiler as a default toolchain > + - name: Set specific C compiler as a default toolchain for cmake > run: | > - echo CC=${CC_NAME} | tee -a $GITHUB_ENV > + echo CMAKE_C_COMPILER=${CC_NAME} | tee -a $GITHUB_ENV > shell: bash > env: > CC_NAME: ${{ inputs.cc_name }} These changes look like debugging of the workflow. Are they necessary? > diff --git a/.github/actions/setup-sanitizers-macos/README.md b/.github/actions/setup-sanitizers-macos/README.md > new file mode 100644 > index 00000000..9ac5eb38 > --- /dev/null > +++ b/.github/actions/setup-sanitizers-macos/README.md > diff --git a/.github/actions/setup-sanitizers-macos/action.yml b/.github/actions/setup-sanitizers-macos/action.yml > new file mode 100644 > index 00000000..9836ea03 > --- /dev/null > +++ b/.github/actions/setup-sanitizers-macos/action.yml > @@ -0,0 +1,73 @@ > + - name: Set CMAKE_BUILD_PARALLEL_LEVEL > + shell: bash > + run: | > + # Set CMAKE_BUILD_PARALLEL_LEVEL environment variable to > + # limit the number of parallel jobs for build/test step. > + NPROC=$(sysctl -n hw.logicalcpu 2>/dev/null) > + echo CMAKE_BUILD_PARALLEL_LEVEL=$(($NPROC + 1)) | tee -a $GITHUB_ENV > + - name: Set MACOSX_DEPLOYMENT_TARGERT > + shell: bash > + run: | > + # Set required MACOSX_DEPLOYMENT_TARGERT environment > + # variable for Makefile.original build. > + # See https://github.com/LuaJIT/LuaJIT/issues/484, > + # https://github.com/LuaJIT/LuaJIT/issues/653. > + echo MACOSX_DEPLOYMENT_TARGET=$(sw_vers -productVersion) | tee -a $GITHUB_ENV > + - name: Install build and test dependencies > + shell: bash > + run: | > + # Install brew using the command from Homebrew repository > + # instructions: https://github.com/Homebrew/install. > + # XXX: 'echo' command below is required since brew > + # installation script obliges the one to enter a newline > + # for confirming the installation via Ruby script. > + brew update || > + echo | /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)" > + # Try to install the packages either upgrade it to avoid > + # of fails if the package already exists with the previous > + # version. > + brew install --force ${CC_FORMULA} cmake make ninja perl || > + brew upgrade ${CC_FORMULA} cmake make ninja perl Can we reuse setup-macos action instead these 3 steps? > diff --git a/.github/workflows/sanitizers-testing.yml b/.github/workflows/sanitizers-testing.yml > index 4bf7d023..fe550b81 100644 > --- a/.github/workflows/sanitizers-testing.yml > +++ b/.github/workflows/sanitizers-testing.yml > + exclude: > + - OS: Linux > + CC: gcc-15 > + - OS: Linux > + CC: clang-21 May we bump gcc and clang instead for Linux as well (I suppose its about time :)) ? I suppose it helps to avoid too many excludes. Also, there is no need for testing these specific compilers instead of modern ones in sanitizers build. If there are any issues when upgrading compilers for sanitizer builds (infrastructure or tests failures) -- feel free to ignore. > -- > 2.49.0 > -- Best regards, Sergey Kaplun