Attention is currently required from: pespin.

laforge has posted comments on this change by pespin. ( 
https://gerrit.osmocom.org/c/libosmo-sigtran/+/40543?usp=email )

Change subject: Differentiate between dynamic and static routes
......................................................................


Patch Set 2:

(1 comment)

File src/ss7_vty.c:

https://gerrit.osmocom.org/c/libosmo-sigtran/+/40543/comment/991a07c6_23202154?usp=email
 :
PS2, Line 521: %
this results in three extraneous blanks at the end of each line, see the 
modififed text output.  Also, given that there's no space before it, the "dyn" 
would be printed immediately aftre the avail/UNAVAIL.

I think `availdyn` or `UNAVAILdyn` is not the intended output?



-- 
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40543?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: Ic6c6b46084a1e4063ebf1f5d13e0e03386bb4c45
Gerrit-Change-Number: 40543
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Fri, 27 Jun 2025 18:39:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to