<p><a href="https://gerrit.osmocom.org/c/pysim/+/23210">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/+/23210/4/pySim-shell.py">File pySim-shell.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/+/23210/4/pySim-shell.py@178">Patch Set #4, Line 178:</a> <code style="font-family:monospace,monospace">                   self._c</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">should we really just ignore erros in reading files? I think we need to check _why_ reading/selectin […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The errors are ignored, but the exception string is printed, so the user can inspect the script afterwards to see which files are problematic and also gets the reason from the execption string. The idea is to read as much data as possible.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This is what I get when a file is not found:<br>6a82: Wrong parameters - File not found</p><p style="white-space: pre-wrap; word-wrap: break-word;">This is what I get when I try to read from a file that requires ADM permissions:<br>Failed to read (offset 0)</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also I am now collecting the exception strings along with the filenames to print a summary at the end, so the errors can not hide anymore somewhere in the script comments.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If there were errors I now also raise an exception that tells the user that some files could not be exported so he can check the logs if required.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/23210">change 23210</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/+/23210"/><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: I82f3ce92cd91a5ed3c4884d62f6b22e9589c8a49 </div>
<div style="display:none"> Gerrit-Change-Number: 23210 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-CC: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 18 Mar 2021 22:03:51 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>