<div dir="ltr"><div>Hello everyone!<br><br></div>I am happy to report, that the GSM 05.03 code migration from OsmoBTS<br>is finished now. For details, please see:<br><br><a href="https://gerrit.osmocom.org/#/c/816/">https://gerrit.osmocom.org/#/c/816/</a><br><br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>With best regards,<br></div><div>Vadim Yanitskiy.<br></div></div></div></div></div></div>
<br><div class="gmail_quote">2016-08-09 0:06 GMT+07:00 Neels Hofmeyr <span dir="ltr"><<a href="mailto:nhofmeyr@sysmocom.de" target="_blank">nhofmeyr@sysmocom.de</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Aug 04, 2016 at 05:50:39PM +0600, Vadim Yanitskiy wrote:<br>
>  * The 'osmo-bts/gsm_data.h', which includes 'openbsc/gsm_data_shared.h'<br>
>      as well.<br>
><br>
> So, there is regarding questions:<br>
><br>
> 3) Which logging category it would be better to use after migration?<br>
> 4) What to do with the 'osmo-bts/gsm_data.h'?<br>
> 5) I don't think, that it's a good way to require something from the<br>
> OpenBSC sources during OsmoBTS build... How can we change it?<br>
<br>
</span>I've also faced the problem twice recently that I effectively can't log from<br>
gsm_data_shared.c, since the logging constants (e.g. DRSL) are defined<br>
separately for openbsc and osmo-bts (can't include both headers...).<br>
<br>
The first time I solved it by returning an error code and code-dup'ing the<br>
error logging in osmo-bts and openbsc (see lchan_lookup() in<br>
openbsc/src/libbsc/abis_rsl.c and osmo-bts/src/common/rsl.c).<br>
<br>
The other time I placed a compile time #warning where I'd have preferred to<br>
post an error log (not committed yet).<br>
<br>
The one advantage of gsm_data_shared.c is that I feel that I can modify that<br>
API without breaking any libosmocore ABI. We can pretty freely drop stuff in<br>
there and re-use across openbsc and osmo-bts that isn't used anywhere else.<br>
<br>
But I find these solutions rather unsatisfactory / ugly and would +1 for<br>
solving the gsm_data_shared.c evil-twin situation. If some of the stuff isn't<br>
worthy of moving to libosmocore, we'd need another tiny osmo-lib?<br>
<span class="HOEnZb"><font color="#888888"><br>
~Neels<br>
<br>
--<br>
- Neels Hofmeyr <<a href="mailto:nhofmeyr@sysmocom.de">nhofmeyr@sysmocom.de</a>>          <a href="http://www.sysmocom.de/" rel="noreferrer" target="_blank">http://www.sysmocom.de/</a><br>
==============================<wbr>==============================<wbr>===========<br>
* sysmocom - systems for mobile communications GmbH<br>
* Alt-Moabit 93<br>
* 10559 Berlin, Germany<br>
* Sitz / Registered office: Berlin, HRB 134158 B<br>
* Geschäftsführer / Managing Directors: Harald Welte<br>
</font></span></blockquote></div><br></div>