[Asterisk-code-review] core/buildsystem: check the actual compiler being version (...asterisk[13])

[Guido Falsi]

Guido Falsi has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/11238


Change subject: core/buildsystem: check the actual compiler being version
......................................................................

core/buildsystem: check the actual compiler being version

Make compiler check use the output of the actual compiler being
used as reported by the CC variable, instead of unconditionally
running the "gcc" binary.

ASTERISK-28374

Change-Id: Icaacf6d93686ad21076878aa1504a23b4fc9d0f4
---
M Makefile.rules
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/38/11238/1

diff --git a/Makefile.rules b/Makefile.rules
index dc23a9e..cfd2c1e 100644
--- a/Makefile.rules
+++ b/Makefile.rules
@@ -68,7 +68,7 @@
 endif
 
 # gcc version 8.2.1 and above must have partial-inlining disabled to avoid documented bug
-GCC_VER_GTE821:=$(shell expr `gcc --version | grep ^gcc | cut -d ' ' -f 3 | sed -e 's/\.\([0-9][0-9]\)/\1/g' -e 's/\.\([0-9]\)/0\1/g' -e 's/^[0-9]\{3,4\}$$/&00/'` \>= 80201)
+GCC_VER_GTE821:=$(shell expr `$(CC) --version | grep ^gcc | cut -d ' ' -f 3 | sed -e 's/\.\([0-9][0-9]\)/\1/g' -e 's/\.\([0-9]\)/0\1/g' -e 's/^[0-9]\{3,4\}$$/&00/'` \>= 80201)
 ifeq ($(GCC_VER_GTE821),1)
     OPTIMIZE+=-fno-partial-inlining
 endif

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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: Icaacf6d93686ad21076878aa1504a23b4fc9d0f4
Gerrit-Change-Number: 11238
Gerrit-PatchSet: 1
Gerrit-Owner: Guido Falsi <madpilot at freebsd.org>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190407/c6896c21/attachment.html>