<p style="white-space: pre-wrap; word-wrap: break-word;">tcpdump runs in the background. With this change it runs after setting the program variables.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Now that you've asked me to look at it again, I realize that I made a mistake here, I thought the script would also check for the xterm program while filling the program variables (see the term() function above). But that is not the case, the term() function gets executed afterwards.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So this commit will only make sense when the part that checks if xterm or urxvt is installed gets executed before starting the tcpdump instances.</p><p style="white-space: pre-wrap; word-wrap: break-word;">(I've added this after the last patch originally, which adds support for osmo-sip-connector, and there I check if the sip server is installed while filling out the program variables.)</p><p style="white-space: pre-wrap; word-wrap: break-word;">Thanks for looking through the patches!</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/11142">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11142">change 11142</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/11142"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-dev </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: If487c357cbae504410545d2ed00d7e22f5e60f4f </div>
<div style="display:none"> Gerrit-Change-Number: 11142 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 28 Sep 2018 10:27:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>