fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/22385 )

Change subject: Introduce NACC support
......................................................................


Patch Set 11:

(1 comment)

https://gerrit.osmocom.org/c/osmo-pcu/+/22385/11/src/nacc_fsm.c
File src/nacc_fsm.c:

https://gerrit.osmocom.org/c/osmo-pcu/+/22385/11/src/nacc_fsm.c@633
PS11, Line 633: connect to CTRL
I am afraid this message may be confusing for the end user, because usually our 
osmo-apps are CTRL servers. Maybe "Failed to establish CTRL (neighbor 
resolution) connection to the BSC r=%s:%u\n"?



--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/22385
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Id35f40d05f3e081f32fddbf1fa34cb338db452ca
Gerrit-Change-Number: 22385
Gerrit-PatchSet: 11
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Comment-Date: Thu, 28 Jan 2021 18:38:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to