Sadly no idea if there is way to workaround current failure:

  /tmp/arangodb-20230630-89543-1vkojor/ArangoDB-3.11.1/lib/Inspection/include/Inspection/Access.h:52:3: error: static assertion failed due to requirement 'detail::IsInspectable()'
    static_assert(detail::IsInspectable<TT, Inspector>());
    ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I'll experiment with downgrading LLVM and/or switching to GCC. Upstream has stated they will not support compiler versions outside of https://github.com/arangodb/arangodb/blob/v3.11.1/VERSIONS

Afterward, we can decide whether to go this route or to deprecate.

Upstream has stated they will not support compiler versions outside of https://github.com/arangodb/arangodb/blob/v3.11.1/VERSIONS

Thing is that it fails across all macOS versions. Using LLVM 12, 14 and 15 based compilers. It's tricky to track down why but I'll see if I can see anything.

Linux error is however caused by us using GCC 10 here, which is too old. We should try 11 again.

Thing is that it fails across all macOS versions. Using LLVM 12, 14 and 15 based compilers. It's tricky to track down why but I'll see if I can see anything.

We are using brewed llvm so LLVM 16. In addition to original reason, upstream has added a version checker that will reject AppleClang (https://github.com/arangodb/arangodb/blob/v3.11.1/cmake/CheckCompilerVersion.cmake)

EDIT: Rather than reject, the logic just doesn't work as MIN_VERSION isn't set.

CMake Warning at cmake/CheckCompilerVersion.cmake:18 (message):
  AppleClang/AppleClang compiler is not supported.
Call Stack (most recent call first):
  CMakeLists.txt:496 (CheckCompilerVersion)


CMake Error at cmake/CheckCompilerVersion.cmake:21 (if):
  if given arguments:

    "(" "CMAKE_C_COMPILER_VERSION" "VERSION_LESS" ")" "OR" "(" "CMAKE_CXX_COMPILER_VERSION" "VERSION_LESS" ")"

  Unknown arguments specified
Call Stack (most recent call first):
  CMakeLists.txt:496 (CheckCompilerVersion)


-- Configuring incomplete, errors occurred!

Linux error is however caused by us using GCC 10 here, which is too old. We should try 11 again.

Yeah. Looks like minimum GCC 11.1 is needed for std::atomic<T>::wait. Will try default GCC 11.3 again, but may need to apply workaround patch if it hits bug.

Thing is that it fails across all macOS versions. Using LLVM 12, 14 and 15 based compilers. It's tricky to track down why but I'll see if I can see anything.

We are using brewed llvm so LLVM 16. In addition to original reason, upstream has added a version checker that will reject AppleClang (https://github.com/arangodb/arangodb/blob/v3.11.1/cmake/CheckCompilerVersion.cmake)

Oh oops misread that.

The script you link technically outputs warnings rather than fails though. But yeah might need to use an older LLVM if Apple Clang doesn't work. Ah I see what you're referring to now - technically a bug really where it should be using elseif.

We should probably just deprecate this formula TBH. It had ~500 downloads in the past month (numbers based on GitHub Packages). I don't think that's worth the amount of CI/maintainer time this formula takes up. (Unless there actually is a maintainer that uses this formula and is willing to work on it.) I'm pretty sure it vendors all sorts of dependencies too.

CC @Homebrew/core for thoughts.

We should probably just deprecate this formula TBH.

That is fine with me. Upstream also sounds like they want us to deprecate/remove it as their response to any build issues is to install their version.

I'm pretty sure it vendors all sorts of dependencies too.

Yeah, it does https://github.com/arangodb/arangodb/tree/devel/3rdParty which is part of reason build fails (V8 and Boost copies have issues with LLVM 16).

Agreed with deprecating it.

Agreed on deprecation 👍🏻

Opened #135907 to deprecate.