Attention is currently required from: pespin. falconia has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/osmo-bts/+/42167?usp=email )
Change subject: RTP: add vty option for ortp vs twrtp selection ...................................................................... Patch Set 5: (2 comments) File src/common/vty.c: https://gerrit.osmocom.org/c/osmo-bts/+/42167/comment/22f674d2_986315d9?usp=email : PS5, Line 434: if (bts->use_twrtp != g_use_twrtp_default) { > tbh, I see no need for the global variable here. […] I know full well that the approach you describe is the standard way used everywhere else. However, the present case is special: * We already plan from the get-go to change this default in the not too distant future. With my implementation only one line of code will need to change at that point, as opposed to having to remember to rework the vty write-out code for the opposite construct. * I plan on a follow-up patch that will introduce `--disable-ortp` configure option, and I hope we both agree that patch should be merge-able well before we reach the point of making twrtp the new default for everyone. Therefore, there will be a period of time when the default is still ortp in the standard build, but the special build with ortp disabled has to default to twrtp as it is the only supported selection. My global variable implementation is already prepared for this eventuality. https://gerrit.osmocom.org/c/osmo-bts/+/42167/comment/a62230e5_d1650e20?usp=email : PS5, Line 1529: vty_out(vty, > I'd say simply add here: […] Considering our already set plan to change the default at some point in the not too distant future, I really feel that both the current setting and the default should be displayed. It is important to know the default in this case, so that the operator can tell which library will be selected with a config file that does not specify `rtp library`. -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/42167?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: Iff4e3a399250c16ba8fe4cb12e4e22f4c6b346ec Gerrit-Change-Number: 42167 Gerrit-PatchSet: 5 Gerrit-Owner: falconia <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Comment-Date: Tue, 24 Feb 2026 13:33:32 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <[email protected]>
