neels submitted this change.

View Change


Approvals: neels: Looks good to me, approved fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified
sccp: set titan.SCCP commit: use fix for localRef == 0

Use this fix in our tests:
https://github.com/osmocom/titan.ProtocolEmulations.SCCP/commit/17a894fc6620a0df11d0d98c897fb66168276b16

This allows osmo-hnbgw to start out with SCCP local reference == 0
without that causing test failure.

'Related' patch below happens to have that effect, and it was *really*
hard to find out why th the tests failed. It seemed like I had messed up
osmo-hnbgw, but then I found out it wasn't my fault at all.

Related: Iea1824f1c586723d989c80a909bae16bd2866e08 osmo-hnbgw
Change-Id: Idb3f2398c934a1d785e8ee7913c12c0f289c1f18
---
M deps/Makefile
1 file changed, 21 insertions(+), 1 deletion(-)

diff --git a/deps/Makefile b/deps/Makefile
index 277bd67..9bdfb8e 100644
--- a/deps/Makefile
+++ b/deps/Makefile
@@ -86,7 +86,7 @@
# file changed.
titan.Libraries.TCCUsefulFunctions_commit= R.35.B-6-gb3687da
titan.ProtocolEmulations.M3UA_commit= f086e78d74defa044d864f17adaad9433fedc961
-titan.ProtocolEmulations.SCCP_commit= R.7.A-11-gd4b7a6d
+titan.ProtocolEmulations.SCCP_commit= 17a894fc6620a0df11d0d98c897fb66168276b16
titan.ProtocolModules.BSSAPP_v7.3.0_commit= R.2.A-4-g20cfaf8
titan.ProtocolModules.BSSGP_v13.0.0_commit= e97d92a8b66bec399babea52f593771b76cb175a
titan.ProtocolModules.BSSMAP_commit= 24f967b2f77bfdd2e6eabf1e82fa76239095f772

To view, visit change 32331. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Idb3f2398c934a1d785e8ee7913c12c0f289c1f18
Gerrit-Change-Number: 32331
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged