render / rpms / edk2

Forked from rpms/edk2 2 months ago
Clone

Blame SOURCES/edk2-BaseTools-header.makefile-remove-c-from-BUILD_CFLAGS.patch

cc9195
From e7091299a079fd0405f01276f35838884b4c06b5 Mon Sep 17 00:00:00 2001
cc9195
From: Laszlo Ersek <lersek@redhat.com>
cc9195
Date: Wed, 29 Aug 2018 17:11:59 +0200
cc9195
Subject: [PATCH 2/7] BaseTools/header.makefile: remove "-c" from BUILD_CFLAGS
cc9195
cc9195
Message-id: <20180829151204.26958-3-lersek@redhat.com>
cc9195
Patchwork-id: 81964
cc9195
O-Subject:  [RHEL8/virt212 edk2 PATCH 2/7] BaseTools/header.makefile: remove "-c"
cc9195
	from BUILD_CFLAGS
cc9195
Bugzilla: 1607906
cc9195
Acked-by: Thomas Huth <thuth@redhat.com>
cc9195
Acked-by: Vitaly Kuznetsov <vkuznets@redhat.com>
cc9195
cc9195
Option "-c" is a mode selection flag (choosing between compiling and
cc9195
linking); it should not be in BUILD_CFLAGS, which applies only to
cc9195
compiling anyway. The compilation rule for C source files, in
cc9195
"footer.makefile", already includes "-c" -- currently we have double "-c"
cc9195
options.
cc9195
cc9195
This patch doesn't change behavior.
cc9195
cc9195
Cc: Liming Gao <liming.gao@intel.com>
cc9195
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
cc9195
Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1540244
cc9195
Contributed-under: TianoCore Contribution Agreement 1.1
cc9195
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
cc9195
Reviewed-by: Liming Gao <liming.gao@intel.com>
cc9195
(cherry picked from commit 03252ae287c4a61983b3793ff71baeabe2ff3df7)
cc9195
---
cc9195
 BaseTools/Source/C/Makefiles/header.makefile | 4 ++--
cc9195
 1 file changed, 2 insertions(+), 2 deletions(-)
cc9195
cc9195
diff --git a/BaseTools/Source/C/Makefiles/header.makefile b/BaseTools/Source/C/Makefiles/header.makefile
cc9195
index db43677..08421ba 100644
cc9195
--- a/BaseTools/Source/C/Makefiles/header.makefile
cc9195
+++ b/BaseTools/Source/C/Makefiles/header.makefile
cc9195
@@ -71,9 +71,9 @@ INCLUDE = $(TOOL_INCLUDE) -I $(MAKEROOT) -I $(MAKEROOT)/Include/Common -I $(MAKE
cc9195
 BUILD_CPPFLAGS = $(INCLUDE) -O2
cc9195
 ifeq ($(DARWIN),Darwin)
cc9195
 # assume clang or clang compatible flags on OS X
cc9195
-BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-deprecated-declarations -Wno-self-assign -Wno-unused-result -nostdlib -c -g
cc9195
+BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-deprecated-declarations -Wno-self-assign -Wno-unused-result -nostdlib -g
cc9195
 else
cc9195
-BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-deprecated-declarations -Wno-stringop-truncation -Wno-restrict -Wno-unused-result -nostdlib -c -g
cc9195
+BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-deprecated-declarations -Wno-stringop-truncation -Wno-restrict -Wno-unused-result -nostdlib -g
cc9195
 endif
cc9195
 BUILD_LFLAGS =
cc9195
 BUILD_CXXFLAGS = -Wno-unused-result
cc9195
-- 
cc9195
1.8.3.1
cc9195