Onnxruntime fix

[chetant]

• Have you followed the guidelines for contributing?
• Have you ensured that your commits follow the commit style guide?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 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 HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

---

Fixes #215955

When running a program compiled with onnxruntime that creates an Ort::Env, we'd get the following schema double registration errors:

Schema error: Trying to register schema with name Abs (domain:  version: 1) from file /tmp/onnx-20250108-35339-a6pxcu/onnx-1.17.0/onnx/defs/math/old.cc line 2743, but it is already registered from file /tmp/onnx-20250108-35339-a6pxcu/onnx-1.17.0/onnx/defs/math/old.cc line 2743

Schema error: Trying to register schema with name Add (domain:  version: 1) from file /tmp/onnx-20250108-35339-a6pxcu/onnx-1.17.0/onnx/defs/math/old.cc line 2627, but it is already registered from file /tmp/onnx-20250108-35339-a6pxcu/onnx-1.17.0/onnx/defs/math/old.cc line 2627

Schema error: Trying to register schema with name And (domain:  version: 1) from file /tmp/onnx-20250108-35339-a6pxcu/onnx-1.17.0/onnx/defs/logical/old.cc line 138, but it is already registered from file /tmp/onnx-20250108-35339-a6pxcu/onnx-1.17.0/onnx/defs/logical/old.cc line 138
...

As per https://onnxruntime.ai/docs/build/dependencies.html#use-preinstalled-packages

Each library can be built in different ways. For example, ONNX Runtime expects ONNX is built with “-DONNX_DISABLE_STATIC_REGISTRATION=ON”. If you have got a prebuilt ONNX library from somewhere, most likely it was not built in this way.

This creates a separate onnx build onnxnostaticreg (with the correct flags) specifically for onnxruntime and updates its dependency.

[carlocab]

Can we just rebuild onnx with this configuration instead?

[chetant]

We can, and it will build, but any program linking with onnx only (and using nodes) will not work correctly, since it will skip over the schema registration at startup.

[cho-m]

We don't want to introduce a conflicting formula that is only used as a dependency for a single formula.

There are 2 options I can think of:

1. Use Fedora's workaround¹ in onnx to specifically to handle onnxruntime.
2. Let onnxruntime build its own copy of onnx and statically link it. In this case, should use protobuf-lite to avoid symbol conflicts in global registry, i.e. -Donnxruntime_USE_FULL_PROTOBUF=OFF

Debian is also trying to resolve this². They did try ONNX_DISABLE_STATIC_REGISTRATION but ended up reverting due to failures running onnx tests.

Footnotes

1. https://src.fedoraproject.org/rpms/onnx/blob/rawhide/f/0004-Add-fixes-for-use-with-onnxruntime.patch ↩

2. https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1092898 ↩