tarantool 2.3.2.1

[avtikhon]

Set stable release to 2.3.2.1 version.

• Have you followed the guidelines for contributing?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

---

[bayandin]

Could you please change commit message to tarantool 2.3.2.1

And there're some references to Homebrew shims directory which don't work outside of Homebrew's superenv:

==> brew audit --skip-style tarantool --online
Error: 1 problem in 1 formula detected
tarantool:
  * Files were found with references to the Homebrew shims directory.
    The offending files are:
      bin/tarantool
      include/tarantool/module.h
==> FAILED

For include/tarantool/module.h it's this line:

#define COMPILER_INFO "@@HOMEBREW_REPOSITORY@@/Library/Homebrew/shims/mac/super/clang @@HOMEBREW_REPOSITORY@@/Library/Homebrew/shims/mac/super/clang++"

[Bo98]

Bypassing a significant portion of the superenv during the actual build kinda sucks. Would modifying COMPILER_INFO be any better?

(We should also wait until icu4c 66.1 build is complete)

[bayandin]

Bypassing a significant portion of the superenv during the actual build kinda sucks. Would modifying COMPILER_INFO be any better?

It seems the only way to do it is to patch ./src/trivia/config.h.cmake, will it be ok?

- #define COMPILER_INFO "@CMAKE_C_COMPILER@ @CMAKE_CXX_COMPILER@"
+ #define COMPILER_INFO "/usr/bin/clang /usr/bin/clang++"

(We should also wait until icu4c 66.1 build is complete)

Ok, will merge after #52481

[Bo98]

will it be ok?

I think so. It's effectively the same as how it replaces those things in the build script anyway. Better than avoiding the superenv IMO.

[BrewTestBot]

@bayandin has triggered a merge.

[bayandin]

Thanks @avtikhon!