File names in #include directives should not be prepended with "src/". The
automake file already adds "/src" to the global INCLUDE path, so prepending with
"src/" is unnecessary:
# This allows #includes to be relative to src/
AM_CPPFLAGS = -I$(top_srcdir)/src
Moreover, prepending with "src/" is inconsistent with the rest of the code and
causes build breaks when attempting to compile this within Google. Updating the
INCLUDE path in the internal Google BUILD file to include the parent directory
can resolve this, but we'd ratter have consistency.
Thanks,
-Ivan
Issue 427003: Prepending with "src/" is undesirabe in #include directives
(Closed)
Created 12 years, 3 months ago by Ivan Penkov
Modified 12 years, 3 months ago
Reviewers: digit, Mark Mentovai, Ted Mielczarek
Base URL: http://google-breakpad.googlecode.com/svn/trunk/src/
Comments: 0