Vadim Yanitskiy has posted comments on this change. ( 
https://gerrit.osmocom.org/13709 )

Change subject: common/oml.c: fix: properly encode NM_ATT_SW_CONFIG
......................................................................


Patch Set 2:

> We have to ensure both old versions of OsmoBTS will work with old and new 
> versions of OpenbSC / OsmoBSC, as well as new versions of OsmoBTS work with 
> old version of OpenBSC/OsmoBSC.

Well, if we merge this change, the old OpenBSC would continue to work with the 
fixed version of OsmoBTS anyway, in the same way as the current OsmoBSC does 
work with the current OsmoBTS. There would be no fundamental breakage. Should I 
add this to the commit message?


--
To view, visit https://gerrit.osmocom.org/13709
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id26104719891944f3e2151df362bd45bb057a9c5
Gerrit-Change-Number: 13709
Gerrit-PatchSet: 2
Gerrit-Owner: Vadim Yanitskiy <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Vadim Yanitskiy <[email protected]>
Gerrit-CC: Harald Welte <[email protected]>
Gerrit-Comment-Date: Sat, 20 Apr 2019 16:18:47 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No

Reply via email to