Tap maintenance and Homebrew development

[timocafe]

Output of brew config

HOMEBREW_VERSION: 4.6.7-61-g9219495
ORIGIN: https://github.com/Homebrew/brew
HEAD: 9219495eb0a66e7559a03759d6c859148c19c617
Last commit: 2 days ago
Branch: main
Core tap JSON: 31 Aug 15:56 UTC
Core cask tap JSON: 31 Aug 15:56 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_FORBID_PACKAGES_FROM_PATHS: set
HOMEBREW_MAKE_JOBS: 8
HOMEBREW_SORBET_RUNTIME: set
Homebrew Ruby: 3.4.5 => /opt/homebrew/Library/Homebrew/vendor/portable-ruby/3.4.5/bin/ruby
CPU: octa-core 64-bit arm_firestorm_icestorm
Clang: 17.0.0 build 1700
Git: 2.39.5 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 8.7.1 => /usr/bin/curl
macOS: 15.6.1-arm64
CLT: 16.4.0.0.1.1747106510
Xcode: 16.4
Rosetta 2: false

Output of brew doctor

Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Unbrewed dylibs were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae and may need to be deleted.

Unexpected dylibs:
  /usr/local/lib/libomp.dylib

Warning: Unbrewed header files were found in /usr/local/include.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae and may need to be deleted.

Unexpected header files:
  /usr/local/include/gmock/*
  /usr/local/include/gtest/*
  /usr/local/include/omp-tools.h
  /usr/local/include/omp.h
  /usr/local/include/ompt.h

Warning: Unbrewed '.pc' files were found in /usr/local/lib/pkgconfig.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae and may need to be deleted.

Unexpected '.pc' files:
  /usr/local/lib/pkgconfig/gmock.pc
  /usr/local/lib/pkgconfig/gmock_main.pc
  /usr/local/lib/pkgconfig/gtest.pc
  /usr/local/lib/pkgconfig/gtest_main.pc

Warning: Unbrewed static libraries were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae and may need to be deleted.

Unexpected static libraries:
  /usr/local/lib/libgmock.a
  /usr/local/lib/libgmock_main.a
  /usr/local/lib/libgtest.a
  /usr/local/lib/libgtest_main.a

Warning: You have pkgconf installed that was built on macOS 14
         but you are running macOS 15.

This can cause issues with packages that depend on system libraries, such as libffi.
To fix this issue, reinstall pkgconf:
  brew reinstall pkgconf

For more information, see: https://github.com/Homebrew/brew/issues/16137
We'd welcome a PR to automatically mitigate this instead of just warning about it.

Warning: Some taps are not on the default git origin branch and may not receive
updates. If this is a surprise to you, check out the default branch with:
  git -C $(brew --repo tewart/local) checkout

Description of issue

The recipe of onnxruntime has two issues:

1. It does not support the COREML understand GPU and NPU for Apple Silicon.
2. A missing option to avoid too much log .

Concerning the first point user already try (like me) to build from the source - Homebrew/homebrew-core#226154. I failed with home-brew but I succeed if I build from scratch (a bit painful)

Concerning the second point (Homebrew/homebrew-core#215955) and an option is needed for cmake:
-DONNX_DISABLE_STATIC_REGISTRATION=ON (conan-io/conan-center-index#17380)

It will be very nice for the community if you could modify the formula. Currently we have only the CPU backend, we are loosing the full potential of Apple silicon. For a fix, I propose:

args = %W[
  -DHOMEBREW_ALLOW_FETCHCONTENT=ON
  -DFETCHCONTENT_FULLY_DISCONNECTED=ON
  -DFETCHCONTENT_TRY_FIND_PACKAGE_MODE=ALWAYS
  -DPython_EXECUTABLE=#{python3}
  -DONNX_CUSTOM_PROTOC_EXECUTABLE=#{Formula["protobuf"].opt_bin}/protoc
  -Donnxruntime_BUILD_SHARED_LIB=ON
  -Donnxruntime_BUILD_UNIT_TESTS=OFF
  -Donnxruntime_GENERATE_TEST_REPORTS=OFF
  -Donnxruntime_RUN_ONNX_TESTS=OFF
  -Donnxruntime_USE_FULL_PROTOBUF=ON
  -DONNX_DISABLE_STATIC_REGISTRATION=ON
]

if Hardware::CPU.arm?
    args << "-Donnxruntime_USE_COREML=ON"
endif

[carlocab]

Pull requests are welcome at https://github.com/Homebrew/homebrew-core/.

Setting ONNX_DISABLE_STATIC_REGISTRATION is unfortunately unlikely to be accepted; see Homebrew/homebrew-core#215960.

However, setting onnxruntime_USE_COREML could probably still be, but that should probably be scoped only to arm64 macOS (and not just any arm64, which is what the current code snippet you show does).

If you need a build that sets ONNX_DISABLE_STATIC_REGISTRATION I suggest using brew extract to create your own version of the formula in your own tap (which you can create with brew tap-new). See man brew for details.

[timocafe]

ok I will try to make my formula work first

[timocafe]

I succeed to run the original formula a bit challenging because Cmake selected "the brew Eigen" although the formula indicates a patch. To make it work I uninstalled the brew Eigen

For now my second problem is on the protobuf, I get this error with protobuf:

Call Stack (most recent call first):
  onnxruntime_providers_coreml.cmake:36 (onnxruntime_protobuf_generate)
  onnxruntime_providers.cmake:148 (include)
  CMakeLists.txt:1868 (include)

I will continue tomorrow