<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/8611">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  George Joseph: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">BuildSystem: For consistency, avoid extra libs to be empty.<br><br>AST_EXT_LIB_CHECK has several optional parameters. When an optional parameter<br>is left empty, [] is used to indicate this. However, this is done in the script<br>./configure only then, when a further parameter is not empty. For example, when<br>no extra libraries are needed to test the checked library, parameter 5 is not<br>mentioned. Except parameter 6 and higher are used, then parameter 5 must be<br>empty.<br><br>However, this general rule was broken<br>* three times for parameter 5 (extra libs) and<br>* three times for parameter 4 (header)<br>as found via the Regular Expression \[\]\). In case of parameter 5, all cases<br>were changed, because that happened for no reason. In case of parameter 4, an<br>[] improves readability actually. Therefore for parameter 4, the only case which<br>did not do it was changed. All this aims to create more consistency: Only do<br>something different if there is a reason to do so.<br><br>Change-Id: I037ef170cf1ad94497151a9ea5071a31c656cafe<br>---<br>M configure.ac<br>1 file changed, 4 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/configure.ac b/configure.ac<br>index e38a368..62f8239 100644<br>--- a/configure.ac<br>+++ b/configure.ac<br>@@ -608,12 +608,12 @@<br> #  * -luuid on Linux<br> #  * -le2fs-uuid on OpenBSD<br> #  * in libsystem on OS X<br>-AST_EXT_LIB_CHECK([LIBUUID], [uuid], [uuid_generate_random], [uuid/uuid.h], [])<br>+AST_EXT_LIB_CHECK([LIBUUID], [uuid], [uuid_generate_random], [uuid/uuid.h])<br> if test "x$LIBUUID_LIB" != "x" ; then<br>   UUID_INCLUDE="$LIBUUID_INCLUDE"<br>   UUID_LIB="$LIBUUID_LIB"<br> else<br>-  AST_EXT_LIB_CHECK([E2FSUUID], [e2fs-uuid], [uuid_generate_random], [uuid/uuid.h], [])<br>+  AST_EXT_LIB_CHECK([E2FSUUID], [e2fs-uuid], [uuid_generate_random], [uuid/uuid.h])<br>   if test "x$E2FSUUID_LIB" != "x" ; then<br>     UUID_INCLUDE="$E2FSUUID_INCLUDE"<br>     UUID_LIB="$E2FSUUID_LIB"<br>@@ -638,7 +638,7 @@<br> fi<br> <br> # See if clock_gettime is in librt<br>-AST_EXT_LIB_CHECK([RT], [rt], [clock_gettime])<br>+AST_EXT_LIB_CHECK([RT], [rt], [clock_gettime], [])<br> <br> AST_EXT_TOOL_CHECK([LIBXML2], [xml2-config], , ,<br>         [#include <libxml/tree.h><br>@@ -2156,7 +2156,7 @@<br> <br> AST_EXT_LIB_CHECK([NEWT], [newt], [newtBell], [newt.h])<br> <br>-AST_EXT_LIB_CHECK([UNIXODBC], [odbc], [SQLConnect], [sql.h], [])<br>+AST_EXT_LIB_CHECK([UNIXODBC], [odbc], [SQLConnect], [sql.h])<br> <br> AST_EXT_LIB_CHECK([OGG], [ogg], [ogg_stream_init], [ogg/ogg.h])<br> <br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/8611">change 8611</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/8611"/><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: merged </div>
<div style="display:none"> Gerrit-Change-Id: I037ef170cf1ad94497151a9ea5071a31c656cafe </div>
<div style="display:none"> Gerrit-Change-Number: 8611 </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: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>