<p style="white-space: pre-wrap; word-wrap: break-word;">I thik we have to be careful here not to damage our OsmoMGW implementation at the sake of interoperability with one broken call agent implementation.  OsmoMGW should continue to support CallIDs of up to 32 hex digits, as in the RFC.  However, libosmo-mgscp-client (and hence OsmoBSC) can simply use shorter CallIDs, which then ends up being compatible with that 3rd party MSC, right?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Changing a common definition shared between our CA and MGW implementations would render our MGW also spec-incompliant...</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/10676">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10676">change 10676</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/10676"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia290c22a91fca0e5aa44515fca6df00064aff100 </div>
<div style="display:none"> Gerrit-Change-Number: 10676 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 29 Aug 2018 07:22:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>