<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7859">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Corey Farrell: 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: Really do not pass unknown-warning options to the compiler.<br><br>When an older GCC version is called with a too new warning option, GCC exited<br>with an error and Asterisk was not built. Therefore, the configure script tests<br>the installed compiler whether it supports that warning option. If not, Asterisk<br>does not pass it to the installed compiler. However, some compilers (like clang)<br>do not exit (error) but give just a warning in such a case. Because the compiler<br>did not exit, Asterisk passed the unknown-warning option.<br><br>ASTERISK-27560<br><br>Change-Id: Ia9b7747f649b27ff5e9f75c3db3fee4fe7a29621<br>---<br>M configure<br>M configure.ac<br>2 files changed, 6 insertions(+), 6 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/configure b/configure<br>index c672df9..8661f31 100755<br>--- a/configure<br>+++ b/configure<br>@@ -18690,7 +18690,7 @@<br> <br> { $as_echo "$as_me:${as_lineno-$LINENO}: checking for -Wdeclaration-after-statement support" >&5<br> $as_echo_n "checking for -Wdeclaration-after-statement support... " >&6; }<br>-if $(${CC} -Wdeclaration-after-statement -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>+if $(${CC} -Wdeclaration-after-statement -Werror -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>      { $as_echo "$as_me:${as_lineno-$LINENO}: result: yes" >&5<br> $as_echo "yes" >&6; }<br>   AST_DECLARATION_AFTER_STATEMENT=-Wdeclaration-after-statement<br>@@ -18742,7 +18742,7 @@<br> <br> { $as_echo "$as_me:${as_lineno-$LINENO}: checking for -Wno-format-truncation" >&5<br> $as_echo_n "checking for -Wno-format-truncation... " >&6; }<br>-if $(${CC} -O2 -Wno-format-truncation -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>+if $(${CC} -Wno-format-truncation -Werror -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>     { $as_echo "$as_me:${as_lineno-$LINENO}: result: yes" >&5<br> $as_echo "yes" >&6; }<br>   AST_NO_FORMAT_TRUNCATION=-Wno-format-truncation<br>@@ -18755,7 +18755,7 @@<br> <br> { $as_echo "$as_me:${as_lineno-$LINENO}: checking for -Wshadow" >&5<br> $as_echo_n "checking for -Wshadow... " >&6; }<br>-if $(${CC} -Wshadow -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>+if $(${CC} -Wshadow -Werror -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>       { $as_echo "$as_me:${as_lineno-$LINENO}: result: yes" >&5<br> $as_echo "yes" >&6; }<br>   AST_SHADOW_WARNINGS=-Wshadow<br>diff --git a/configure.ac b/configure.ac<br>index 0a4fb47..f0a978e 100644<br>--- a/configure.ac<br>+++ b/configure.ac<br>@@ -1224,7 +1224,7 @@<br> AC_SUBST(AST_UNDEFINED_SANITIZER)<br> <br> AC_MSG_CHECKING(for -Wdeclaration-after-statement support)<br>-if $(${CC} -Wdeclaration-after-statement -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>+if $(${CC} -Wdeclaration-after-statement -Werror -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>        AC_MSG_RESULT(yes)<br>    AST_DECLARATION_AFTER_STATEMENT=-Wdeclaration-after-statement<br> else<br>@@ -1264,7 +1264,7 @@<br> AC_SUBST(AST_NO_STRICT_OVERFLOW)<br> <br> AC_MSG_CHECKING(for -Wno-format-truncation)<br>-if $(${CC} -O2 -Wno-format-truncation -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>+if $(${CC} -Wno-format-truncation -Werror -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>   AC_MSG_RESULT(yes)<br>    AST_NO_FORMAT_TRUNCATION=-Wno-format-truncation<br> else<br>@@ -1274,7 +1274,7 @@<br> AC_SUBST(AST_NO_FORMAT_TRUNCATION)<br> <br> AC_MSG_CHECKING(for -Wshadow)<br>-if $(${CC} -Wshadow -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>+if $(${CC} -Wshadow -Werror -S -o /dev/null -xc /dev/null > /dev/null 2>&1); then<br>     AC_MSG_RESULT(yes)<br>    AST_SHADOW_WARNINGS=-Wshadow<br> else<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7859">change 7859</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/7859"/><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: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ia9b7747f649b27ff5e9f75c3db3fee4fe7a29621 </div>
<div style="display:none"> Gerrit-Change-Number: 7859 </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: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>