Change in osmocom-bb[master]: trx_toolkit/{ctrl, data}_if.py: add init log message

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Vadim Yanitskiy gerrit-no-reply at lists.osmocom.org
Mon Jan 14 20:27:43 UTC 2019


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

Change subject: trx_toolkit/{ctrl,data}_if.py: add init log message
......................................................................


Patch Set 3:

> it might avoid confusion if we don't call it CTRL in Osmocom... if
 > not in the code, then maybe at least in logging? TRXCTRL? TRXCTL?

TBH, I don't see any confusion with libosmoctrl's CTRL interface (which could be also named in more concrete way, e.g. OCTRL) as TRX toolkit is not using this library and will never do.

But I understand that some users, who are just starting to use this framework, may confuse this control interface with libosmoctrl's one.

So, ok, let's stick to TRXC / TRXD at least in logging.
I'll do this in a separate change.


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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1a2caf6999ed4f33df76328e48ff5076d166d9fe
Gerrit-Change-Number: 12547
Gerrit-PatchSet: 3
Gerrit-Owner: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Comment-Date: Mon, 14 Jan 2019 20:27:43 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190114/c23dab91/attachment.htm>


More information about the gerrit-log mailing list