Attention is currently required from: pespin.

falconia has posted comments on this change by falconia. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/42164?usp=email )

Change subject: RTP: add vty configuration for twjit
......................................................................


Patch Set 2:

(1 comment)

File src/common/vty.c:

https://gerrit.osmocom.org/c/osmo-bts/+/42164/comment/81dfc1e2_1bea9b78?usp=email
 :
PS2, Line 494:
> shouldn't vty indentation be "  " here (2space)?
No, it is correct - there is one more space added inside the library. (The 
original version emitted the `twjit` line inside the library too.) I also 
tested this code on a sysmoBTS, and the output of `show running-config` looks 
correct, all indentation is as it should be.



--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/42164?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: I9ccd060fce8416e01356fccd9ae465a7298a9ce5
Gerrit-Change-Number: 42164
Gerrit-PatchSet: 2
Gerrit-Owner: falconia <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Mon, 23 Feb 2026 09:00:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <[email protected]>

Reply via email to