<p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/24282">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/24282/1/src/osmo_gsm_tester/obj/enb_srs.py">File src/osmo_gsm_tester/obj/enb_srs.py:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/24282/1/src/osmo_gsm_tester/obj/enb_srs.py@101">Patch Set #1, Line 101:</a> <code style="font-family:monospace,monospace">        </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">trailing whitespace</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/24282/1/src/osmo_gsm_tester/obj/enb_srs.py@107">Patch Set #1, Line 107:</a> <code style="font-family:monospace,monospace">                return</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">return not needed here after raise</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/24282/1/src/osmo_gsm_tester/obj/enb_srs.py@110">Patch Set #1, Line 110:</a> <code style="font-family:monospace,monospace">            srsENB.REMOTE_DIR = path</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">so you are changing a static field here? meaning you are changing the path for ALL instances. Not sure what you are trying to achieve here but doesn't look like the proper way of doing it.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/24282">change 24282</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/c/osmo-gsm-tester/+/24282"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-tester </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I77a56835a4054b86d1d3f7da823a013b3370f767 </div>
<div style="display:none"> Gerrit-Change-Number: 24282 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alejandro Leal <alejandro.leal@srs.io> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 20 May 2021 12:55:36 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>