Attention is currently required from: falconia, jolly.

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

Change subject: bring twjit into libosmo-netif ......................................................................

Patch Set 5:

(1 comment)

File src/twjit.c:

https://gerrit.osmocom.org/c/libosmo-netif/+/39280/comment/47567b20_09a9b82c... : PS2, Line 504: rtph = osmo_rtp_get_hdr(msg);

Fair enough: I'll add this option to the next iteration of the present patchset. […]

@falcon@freecalypso.org yes several people I know, not only you, (eg. me and @vyanitskiy@sysmocom.de) are happy to get rid of ortp dependency since it creates a lot of hassle to maintain, and yes, I was expecting you to submit a patch against osmo-bts to drop ortp and use twrtp instead (tbh I don't recall now if you already submitted that since a lot of time has happened?).

Now I hope you understand better my point regarding support M bit. If we find more not envisioned problematic interop scenarios later on I'm sure we can extend twrtp out of that "closed-minded approach of emulating TDM" in order to get those fixed/improved. AFAIK nobody her anyway looked at full internals of ortp to make sure everything is 100% correct there.