[Asterisk-code-review] Misc fixes in build_tools and sound_tools (...repotools[master])

[George Joseph]

George Joseph has submitted this change and it was merged. ( https://gerrit.asterisk.org/c/repotools/+/11218 )

Change subject: Misc fixes in build_tools and sound_tools
......................................................................

Misc fixes in build_tools and sound_tools

build_tools:  Fixed install target prerequisites in
Makefile.product_rules

sound_tools:  Added .gitignore files for build products in
audiofilter and makeg722.

Change-Id: Ic97fc337dff63fc2d9babe60cadc0bd9b6d9af24
---
M build_tools/Makefile.product_rules
A sound_tools/audiofilter/.gitignore
A sound_tools/makeg722/.gitignore
3 files changed, 9 insertions(+), 2 deletions(-)

Approvals:
  Benjamin Keith Ford: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved; Approved for Submit



diff --git a/build_tools/Makefile.product_rules b/build_tools/Makefile.product_rules
index 726fe75..cd6537c 100644
--- a/build_tools/Makefile.product_rules
+++ b/build_tools/Makefile.product_rules
@@ -369,8 +369,8 @@
 	-@$(MAKE) --quiet --no-print-directory -C asterisk distclean &>/dev/null
 
 .PHONY: install everything
-install:: $(MODULE_NAME).so $(MODULE_EXTRA_LIBRARIES)
-	$(INSTALL) -m 755 -t $(DESTDIR)$(ASTLIBDIR)/modules/ $^
+install:: $(MODULE_LIBS)
+	$(INSTALL) -p -m 755 -t $(DESTDIR)$(ASTLIBDIR)/modules/ $^
 
 everything: superclean tarballs
 
diff --git a/sound_tools/audiofilter/.gitignore b/sound_tools/audiofilter/.gitignore
new file mode 100644
index 0000000..bfd5cf3
--- /dev/null
+++ b/sound_tools/audiofilter/.gitignore
@@ -0,0 +1,2 @@
+audiofilter
+**.o
diff --git a/sound_tools/makeg722/.gitignore b/sound_tools/makeg722/.gitignore
new file mode 100644
index 0000000..697fafa
--- /dev/null
+++ b/sound_tools/makeg722/.gitignore
@@ -0,0 +1,5 @@
+makeg722
+makesln8
+makesln16
+sln8tog722
+**.o

-- 
To view, visit https://gerrit.asterisk.org/c/repotools/+/11218
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: repotools
Gerrit-Branch: master
Gerrit-Change-Id: Ic97fc337dff63fc2d9babe60cadc0bd9b6d9af24
Gerrit-Change-Number: 11218
Gerrit-PatchSet: 2
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190403/d69b3303/attachment.html>