[Asterisk-code-review] Makefile: Use basename in a POSIX-compliant way. (asterisk[16])

Sean Bright asteriskteam at digium.com
Thu Oct 7 12:52:04 CDT 2021


Sean Bright has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/16570 )


Change subject: Makefile: Use basename in a POSIX-compliant way.
......................................................................

Makefile: Use basename in a POSIX-compliant way.

If you aren't using GNU coreutils, chances are that your basename
doesn't know about the -s argument. Luckily for us, basename does what
we need it do even without the -s argument.

Change-Id: I8b81a429bb037b997ee6640ff8a2b5e860962bb7
---
M Makefile
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/70/16570/1

diff --git a/Makefile b/Makefile
index 53a99bb..3a2eaf5 100644
--- a/Makefile
+++ b/Makefile
@@ -493,7 +493,7 @@
 			MODULEINFO=$$($(AWK) -f build_tools/get_moduleinfo $$i) ; \
 			if [ -n "$$MODULEINFO" ] ; \
 			then \
-				echo "<module language=\"en_US\" name=\"`$(BASENAME) -s .c $$i`\">" >> $@ ; \
+				echo "<module language=\"en_US\" name=\"`$(BASENAME) $$i .c`\">" >> $@ ; \
 				echo "$$MODULEINFO" >> $@ ; \
 				echo "</module>" >> $@ ; \
 			fi ; \

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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I8b81a429bb037b997ee6640ff8a2b5e860962bb7
Gerrit-Change-Number: 16570
Gerrit-PatchSet: 1
Gerrit-Owner: Sean Bright <sean at seanbright.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20211007/116a5993/attachment.html>


More information about the asterisk-code-review mailing list