<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7939">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  George Joseph: Looks good to me, but someone else must approve
  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: Ia9d148e689c173df4e91699113605dab2de36038<br>---<br>M third-party/pjproject/patches/user.mak<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/third-party/pjproject/patches/user.mak b/third-party/pjproject/patches/user.mak<br>index dafb259..b7eab52 100644<br>--- a/third-party/pjproject/patches/user.mak<br>+++ b/third-party/pjproject/patches/user.mak<br>@@ -1,4 +1,4 @@<br> <br>-NUBSV := $(shell gcc -Wno-unused-but-set-variable -o /dev/null -xc -c - </dev/null 2>/dev/null && echo -Wno-unused-but-set-variable)<br>+NUBSV := $(shell ${CC} -Wno-unused-but-set-variable -Werror -o /dev/null -xc -c - </dev/null 2>/dev/null && echo -Wno-unused-but-set-variable)<br> <br> CFLAGS += -fPIC $(NUBSV) -Wno-unused-variable -Wno-unused-label -Wno-unused-function -Wno-strict-aliasing<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7939">change 7939</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/7939"/><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: Ia9d148e689c173df4e91699113605dab2de36038 </div>
<div style="display:none"> Gerrit-Change-Number: 7939 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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-Reviewer: Joshua Colp <jcolp@digium.com> </div>