Multiplatform Makefile Update
authorAndrew Latham <lathama@gmail.com>
Sat, 13 Oct 2012 17:38:41 +0000 (17:38 +0000)
committerAndrew Latham <lathama@gmail.com>
Sat, 13 Oct 2012 17:38:41 +0000 (17:38 +0000)
Paul Belanger pointed out that using sed in the Makefile is an issue with multiple platforms.  We are cleaning up the Doxygen config as a following step so I just switched the sed inplace changes to be an echo append instead.

(issue ASTERISK-20259)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@374963 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Makefile

index 9b2dc22..49b6f9f 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -744,11 +744,11 @@ ifeq ($(DOT),:)
        @echo "DOT is not installed. Doxygen will not produce any diagrams. Please install and re-run the configuration script."
 else
        # Enable DOT
-       @sed -i "/^HAVE_DOT/s/NO/YES/" contrib/asterisk-ng-doxygen
+       @echo "HAVE_DOT = YES" >> contrib/asterisk-ng-doxygen
 endif
        # Set Doxygen PROJECT_NUMBER variable
 ifneq ($(ASTERISKVERSION),UNKNOWN__and_probably_unsupported)
-       @sed -i "/^PROJECT_NUMBER/s/PROJECT_NUMBER.*/PROJECT_NUMBER = "$(ASTERISKVERSION)"/" contrib/asterisk-ng-doxygen
+       @echo "PROJECT_NUMBER = $(ASTERISKVERSION)" >> contrib/asterisk-ng-doxygen
 else
        echo "Asterisk Version is unknown, not configuring Doxygen PROJECT_NUMBER."
 endif