<p>Alexander Traud has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/8609">View Change</a></p><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>* four 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, 5 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/09/8609/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/configure.ac b/configure.ac<br>index 7dc3b76..de34c42 100644<br>--- a/configure.ac<br>+++ b/configure.ac<br>@@ -645,12 +645,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>@@ -675,7 +675,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>@@ -2193,10 +2193,10 @@<br> # script bug which does not find the ldns library.  The bug is fixed in<br> # v1.4.22 but that version is not easily detectable.<br> #<br>-AST_EXT_LIB_CHECK([UNBOUND], [unbound], [ub_ctx_delete], [unbound.h], [])<br>+AST_EXT_LIB_CHECK([UNBOUND], [unbound], [ub_ctx_delete], [unbound.h])<br> AST_C_DECLARE_CHECK([UNBOUND_CONST_PARAMS], [UNBOUND_VERSION_MAJOR], [unbound.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/8609">change 8609</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/8609"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I037ef170cf1ad94497151a9ea5071a31c656cafe </div>
<div style="display:none"> Gerrit-Change-Number: 8609 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>