<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 4:</p><p style="white-space: pre-wrap; word-wrap: break-word;">How do tests fail? Why not fixing the tests instead of removing unlink()?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">OK I have figured out what the test is doing.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The problem is that unlink fails if the socket path does not yet exist.<br>So we will probably need to catch ENOENT and ignore it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">stsp@fintan:~/osmo/osmo-bsc$ osmo-bsc -r tmp_dummy_sock -c doc/examples/osmo-bsc/osmo-bsc.cfg <br>logging level cc (debug|info|notice|error|fatal)<br>logging level mgcp (debug|info|notice|error|fatal)<br>% dyn_ts_allow_tch_f is deprecated, rather use msc/codec-list to pick codecs<br>T3212 = 5 * 6min (Periodic Location Update timer, sent to MS (1 = 6 minutes))<br><0013> telnet_interface.c:104 Available via telnet 127.0.0.1 4242<br><0015> input/ipaccess.c:844 enabling ipaccess BSC mode on 0.0.0.0 with OML 3002 and RSL 3003 TCP ports<br><001a> control_if.c:911 CTRL at 127.0.0.1 4249<br><0015> bsc_rf_ctrl.c:460 Could not unlink socket path tmp_dummy_sock: 2/No such file or directory<br>Failed to create the RF service.<br></p><p><a href="https://gerrit.osmocom.org/11782">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11782">change 11782</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/11782"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia0c873da305cbb47aef0562f61ec21057363f294 </div>
<div style="display:none"> Gerrit-Change-Number: 11782 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 19 Nov 2018 11:41:54 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>