Attention is currently required from: fixeria.

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

Change subject: VTY: rename 'sctp-role' to 'transport-role', add an alias
......................................................................


Patch Set 4:

(1 comment)

File tests/vty/ss7_asp_test.vty:

https://gerrit.osmocom.org/c/libosmo-sccp/+/35978/comment/875df367_7eddba69
PS2, Line 425:   role sg
             :   transport-role ser
> In general, the same problem can occur even if we keep the old commands 
> unchanged: we oftentimes add […]
This is one of the reasons why I've always been advocating for the 'only save 
those parameters which differ from the default' strategy.  This way any new 
command would not change the 'save' output as long as the compiled-in default 
value is used. As far as I recall, some other developers don't share that view 
and believe it's best to save even default values - at the cost of the problems 
you described.



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

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: Iab6c898181d79a5ed2bea767ee90e55bc3af16a5
Gerrit-Change-Number: 35978
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Tue, 20 Feb 2024 19:06:36 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>
Comment-In-Reply-To: fixeria <[email protected]>
Gerrit-MessageType: comment

Reply via email to