<p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21512">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/osmo-gsm-tester/+/21512/2/src/osmo_gsm_tester/core/process.py">File src/osmo_gsm_tester/core/process.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/+/21512/2/src/osmo_gsm_tester/core/process.py@388">Patch Set #2, Line 388:</a> <code style="font-family:monospace,monospace">    def get_stdout_mark(self):</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Using this with get_stdout looks like a source of all Earth problem. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">i don't see any race, really. it's not like the stdout is being "piped away", this is process.py returning all of the stdout recorded so far, and it is constantly growing. The returned mark is just an integer that remains valid until the process object is discarded, no matter how many marks are requested. Am I missing something?</p><p style="white-space: pre-wrap; word-wrap: break-word;">But i don't actually use the get_stdout* and get_stderr* variants, i end up using only get_output_mark(which='stderr') and grep_output(which='stderr') in my code. So I see unused API creep.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I never heard of "source of all Earth"?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21512">change 21512</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/+/21512"/><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: Ib6569f7486e9d961bd79a5f24232e58d053667a1 </div>
<div style="display:none"> Gerrit-Change-Number: 21512 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </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: Sun, 06 Dec 2020 22:17:36 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>