<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8444">View Change</a></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>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/8444/1/configure.ac">File configure.ac:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8444/1/configure.ac@2150">Patch Set #1, Line 2150:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;"># possible places for OSS definitions<br># Linux<br>AST_EXT_LIB_CHECK([OSS], [ossaudio], [], [linux/soundcard.h])<br># NetBSD: Search for a symbol like _oss_ioctl, to link its OSS Audio Emulation Library.<br>AST_EXT_LIB_CHECK([OSS], [ossaudio], [_oss_ioctl], [sys/soundcard.h])<br># FreeBSD<br>AST_EXT_LIB_CHECK([OSS], [ossaudio], [], [sys/soundcard.h])<br># OpenBSD<br>AST_EXT_LIB_CHECK([OSS], [ossaudio], [oss_ioctl_mixer], [soundcard.h])<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">No, that emulation API might exist on other platforms as well.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Every compile/link test makes the configure script take longer.  If other platforms have this problem another patch can be done to include that platform for the additional checks.  We don't need to make the configure script take even longer because of some theoretical non-Linux platform.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Keep in mind this (and many others) are optional libraries, so checking for the Linux location first does not always eliminate the additional checks.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8444">change 8444</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/8444"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia230d075218551a2f53171c1b37c211914d03268 </div>
<div style="display:none"> Gerrit-Change-Number: 8444 </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: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 11 Mar 2018 18:34:22 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>