<p style="white-space: pre-wrap; word-wrap: break-word;">See comment, unfortunately this interferes with the tests.</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/c/pysim/+/23720">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/pysim/+/23720/1/pySim/transport/__init__.py">File pySim/transport/__init__.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/pysim/+/23720/1/pySim/transport/__init__.py@185">Patch Set #1, Line 185:</a> <code style="font-family:monospace,monospace">                     print("Using PC/SC reader interface (dev_number=%u)" % opts.pcsc_dev)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Since this line also ends up in the results we use to compare when we are running the tests this is is a problem. We had this line before, but we decided to remove it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">(We could also keep it but then we need to regenerate the .ok files and take care that the reader enumeration keeps always the same. Also we must not swap cards then.)</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/23720">change 23720</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/pysim/+/23720"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: pysim </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I87fd934243bc3556fd6db0ce6d219c22a60c2db4 </div>
<div style="display:none"> Gerrit-Change-Number: 23720 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 14 Apr 2021 10:45:24 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>