Attention is currently required from: jolly.
laforge has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-msc/+/33512 )
Change subject: ASCI: Add VTY to configure GCR (Group Call Register)
......................................................................
Patch Set 12:
(3 comments)
File src/libmsc/asci_vty.c:
https://gerrit.osmocom.org/c/osmo-msc/+/33512/comment/aa36a2b0_914b9692
PS10, Line 38: DEFUN(vgcs_call, vgcs_call_cmd,
This is used to initiate a voice group call from the
VTY. […]
Done
https://gerrit.osmocom.org/c/osmo-msc/+/33512/comment/55d175ba_6c2fdb5c
PS10, Line 219: "mute-talker", "Mute talker's downlink")
This is not a dispatcher function. Unmuting talker is
used for testing a call with a single phone. […]
if it's only for testing then
it probably should be HIDDEN and maybe also not saved in the config file?
https://gerrit.osmocom.org/c/osmo-msc/+/33512/comment/ce900301_0e7429cc
PS10, Line 261: "no cell POINT_CODE [<0-65535>]", NO_STR
"Remove BSS/cell from current group\n" PC_ID_STR)
It is default 3.8.8 format. I changed the description
accordingly.
as all our libosmo-sigtran code has user-configurable point code
format, I think the same point code format should be used here. Otherwise it's
inconsistent to the user. The user doesn't know whihc part of the config is parsed by
a library or the application. He'd just be presented with s ome config lines that use
his specified format, while others require a hard-coded format.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-msc/+/33512
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I5bd034a62fc8b483f550d29103c2f7587198f590
Gerrit-Change-Number: 33512
Gerrit-PatchSet: 12
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Wed, 05 Jul 2023 13:39:29 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: jolly <andreas(a)eversberg.eu>
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment