<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/9233">View Change</a></p><p>Patch set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">><br>> rawsocket typically requires root, it's not really a dependency<br>> that can be missing.  The 'dependency' actually just verifies<br>that<br>> `socket.socket(socket.AF_PACKET, socket.SOCK_RAW)` passes.  See<br>> lib/python/asterisk/test_config.py Dependency.depend_rawsocket<br>for<br>> the test code.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I guess then it's an information problem. By adding the custom<br>dependency and not running as root the test just spits out<br>"Dependency: rawsocket - False". Without this patch you get a much<br>more informative error: "Can't create raw IPv4 socket (need to run<br>as root?)"</p><p style="white-space: pre-wrap; word-wrap: break-word;">Perhaps a print or log statement saying why the dependency failed<br>would make sense?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think the dependency checking process can produce custom messages like that.  The main point is that this patch causes the test to be skipped instead of failing, working towards making it possible to run the testsuite without root and get 0% failure.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/9233">change 9233</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.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.asterisk.org/9233"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I9d4307d8e54f36e9c59dfc5096accea321cff7d6 </div>
<div style="display:none"> Gerrit-Change-Number: 9233 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 21 Jun 2018 15:01:04 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>