Does every cc support dumpversion? thanks, ray ________________________________ From: devel@edk2.groups.io on behalf of Yuwei Chen Sent: Wednesday, July 20, 2022 8:21:45 AM To: devel@edk2.groups.io Cc: Feng, Bob C ; Gao, Liming Subject: [edk2-devel] [Patch V2 1/1] BaseTools: remove directly calling for gcc command In some environment, the directly calling for gcc is not supported. This patch fixes the issue. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/C/DevicePath/GNUmakefile | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/BaseTools/Source/C/DevicePath/GNUmakefile b/BaseTools/Source/C/DevicePath/GNUmakefile old mode 100644 new mode 100755 index c217674345b1..5944e80bd2b4 --- a/BaseTools/Source/C/DevicePath/GNUmakefile +++ b/BaseTools/Source/C/DevicePath/GNUmakefile @@ -13,12 +13,15 @@ OBJECTS = DevicePath.o UefiDevicePathLib.o DevicePathFromText.o DevicePathUtili include $(MAKEROOT)/Makefiles/app.makefile -GCCVERSION = $(shell gcc -dumpversion | awk -F'.' '{print $$1}') +GCCVERSION=0 +ifneq ($(CXX), gcc) + GCCVERSION = $(shell $(CXX) -dumpversion | awk -F'.' '{print $$1}') +endif ifneq ("$(GCCVERSION)", "5") -ifneq ($(CXX), llvm) -# gcc 12 trips over device path handling -BUILD_CFLAGS += -Wno-error=stringop-overflow -endif + ifneq ($(CXX), llvm) + # gcc 12 trips over device path handling + BUILD_CFLAGS += -Wno-error=stringop-overflow + endif endif LIBS = -lCommon -- 2.27.0.windows.1