<p>George Joseph <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/8465">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
  George Joseph: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">BuildSystem regression: Fix errors reported by clean targets.<br><br>Doing a 'make clean', 'make distclean', or 'make dist-clean' gets errors<br>about an invalid shell option: "/bin/sh: 0: Illegal option -".<br><br>The clean targets do not include the makeopts file which defines GREP and<br>LDCONFIG because the file may not exist and the distclean/dist-clean<br>targets will delete it anyway.<br><br>ASTERISK-27715<br><br>Change-Id: I33d40acdb03862bc89aeb6fb1ff497894a8ea7f5<br>---<br>M Makefile<br>1 file changed, 15 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/Makefile b/Makefile<br>index c0138db..b7d9814 100644<br>--- a/Makefile<br>+++ b/Makefile<br>@@ -240,7 +240,9 @@<br>   _ASTCFLAGS+=-Wcast-align -DSOLARIS -I../include/solaris-compat -I/opt/ssl/include -I/usr/local/ssl/include -D_XPG4_2 -D__EXTENSIONS__<br> endif<br> <br>-ifneq ($(GREP),:)<br>+ifeq ($(GREP),)<br>+else ifeq ($(GREP),:)<br>+else<br>   ASTERISKVERSION:=$(shell GREP=$(GREP) AWK=$(AWK) GIT=$(GIT) build_tools/make_version .)<br> endif<br> ifneq ($(AWK),)<br>@@ -468,7 +470,9 @@<br>          $(INSTALL) -m 644 $$x "$(DESTDIR)$(ASTDATADIR)/rest-api" ; \<br>        done<br> <br>-ifneq ($(GREP),:)<br>+ifeq ($(GREP),)<br>+else ifeq ($(GREP),:)<br>+else<br>   XML_core_en_US = $(foreach dir,$(MOD_SUBDIRS),$(shell $(GREP) -l "language=\"en_US\"" $(dir)/*.c $(dir)/*.cc 2>/dev/null))<br> endif<br> <br>@@ -487,7 +491,9 @@<br>    @echo<br>         @echo "</docs>" >> $@<br> <br>-ifneq ($(GREP),:)<br>+ifeq ($(GREP),)<br>+else ifeq ($(GREP),:)<br>+else<br>   XML_full_en_US = $(foreach dir,$(MOD_SUBDIRS),$(shell $(GREP) -l "language=\"en_US\"" $(dir)/*.c $(dir)/*.cc 2>/dev/null))<br> endif<br> <br>@@ -631,7 +637,9 @@<br>     fi<br> <br> ld-cache-update:<br>-ifneq ($(LDCONFIG),:)<br>+ifeq ($(LDCONFIG),)<br>+else ifeq ($(LDCONFIG),:)<br>+else<br> ifeq ($(DESTDIR),)  # DESTDIR means binary archive creation; ldconfig should be run on postinst<br>   @if [ $$(id -u) -eq 0 ] ; then \<br>              $(LDCONFIG) "$(ASTLIBDIR)/" ; \<br>@@ -985,7 +993,9 @@<br>        rm -f $(DESTDIR)$(DAHDI_UDEV_HOOK_DIR)/40-asterisk<br> endif<br>    $(MAKE) -C sounds uninstall<br>-ifneq ($(LDCONFIG),:)<br>+ifeq ($(LDCONFIG),)<br>+else ifeq ($(LDCONFIG),:)<br>+else<br>  $(LDCONFIG) "$(ASTLIBDIR)/" || :<br> endif<br> <br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/8465">change 8465</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/8465"/><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: I33d40acdb03862bc89aeb6fb1ff497894a8ea7f5 </div>
<div style="display:none"> Gerrit-Change-Number: 8465 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.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>