Attention is currently required from: pespin.
msuraev has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-msc/+/28849 )
Change subject: libosmonetify SMPP
......................................................................
Patch Set 54:
(1 comment)
This change is ready for review.
File src/libsmpputil/smpp_vty.c:
https://gerrit.osmocom.org/c/osmo-msc/+/28849/comment/0c26441a_f6b9ed37
PS49, Line 190: vty_out(vty, " local-tcp-port %u%s", smsc->listen_port ?
smsc->listen_port : SMPP_PORT, VTY_NEWLINE);
not sure how that would break VTY tests. If user
doesn't set it, then default port should be used.
Fine by me. Add me as reviewer
if you decide to implement this.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-msc/+/28849
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Idc2e60af1010783e555e61b114ae61f55a89d890
Gerrit-Change-Number: 28849
Gerrit-PatchSet: 54
Gerrit-Owner: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 27 Oct 2022 16:07:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Comment-In-Reply-To: msuraev <msuraev(a)sysmocom.de>
Gerrit-MessageType: comment