Add -DTHIRD_PARTY to CFLAGS so as to configure system headers.
EC includes such as common.h are required for CONFIG_* defines,
which configure system headers, e.g. assert.h includes "common.h".
This means that third_party code would need broader include paths
beyond -Ibuiltin. This is not desirable.
This change adds a -DTHIRD_PARTY to the tpm2 library and works
in tandem with a corresponding change to builtin/ that
guards against inclusion of platform headers.
The build environment will have to pass appropriate
-DCONFIG_* via CFLAGS to third_party libraries in order
to get the desired platform behavior.
BRANCH=none
BUG=chrome-os-partner:43025,chrome-os-partner:47524,chrome-os-partner:53782
TEST=compilation succeeds
Change-Id: I63300b0b200bb55a583a85a5b1f8aabc9dd5e56c
Signed-off-by: nagendra modadugu <ngm@google.com>
Reviewed-on: https://chromium-review.googlesource.com/347412
Commit-Ready: Nagendra Modadugu <ngm@google.com>
Tested-by: Nagendra Modadugu <ngm@google.com>
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
diff --git a/Makefile b/Makefile
index 27f63ec..ffc90ce 100644
--- a/Makefile
+++ b/Makefile
@@ -20,6 +20,7 @@
CFLAGS += -Iinclude
CFLAGS += -Wall -Werror
+CFLAGS += -DTHIRD_PARTY
OBJS = $(patsubst %.c,$(obj)/%.o,$(SOURCES))
DEPS = $(patsubst %.c,$(obj)/%.d,$(SOURCES))
