Change in gr-gsm[master]: apps/grgsm_trx: migrate from getopt to argparse

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
Tue Sep 11 07:56:15 UTC 2018


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

Change subject: apps/grgsm_trx: migrate from getopt to argparse
......................................................................


Patch Set 1:

> some of my changes were just thrown away without explanation

The explanation is simple: both are not related to argparse itself.
I prefer to have atomic changes, so if some day we will have to
revert this particular commit, we wouldn't lose the changes you
mentioned below.

So, let's keep this change as-is, because it does exactly what is
stated in the commit message. While renaming variables and changing
class definitions for sure can (and I think should) be done in
separate changes ;)


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

Gerrit-Project: gr-gsm
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I24a17b4cd44db0ce95a19d7470f4f09f3c85a26d
Gerrit-Change-Number: 10832
Gerrit-PatchSet: 1
Gerrit-Owner: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: Piotr Krysik <ptrkrysik at gmail.com>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Comment-Date: Tue, 11 Sep 2018 07:56:15 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180911/f125e2d4/attachment.htm>


More information about the gerrit-log mailing list