Attention is currently required from: fixeria, laforge.

pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/35502?usp=email )

Change subject: trau/osmo_ortp: support building without OrtpMemoryFunctions API ......................................................................

Patch Set 1:

(1 comment)

Patchset:

PS1:

I am saying that for libosmo-abis, bcmtoolbox is not a direct (but indirect or implicit) dependency, pulled by ortp.

If it's in the public header, imho it's not an indirect or implicit dependency, it's explicitly part of the ortp API. Otherwise it would be hidden in private headers or .c files.

This is why I don't think it's a good idea to use API of bcmtoolbox.

Agree.

I am inclined to remove this talloc-to-ortp integration completely

I don't have a strong opinion there, so as you wish.