-
Notifications
You must be signed in to change notification settings - Fork 111
Open
Description
We tend to abuse the includes
attribute of cc_library
: https://docs.bazel.build/versions/master/be/c-cpp.html#cc_library.includes. Using includes
impact all the dependent rules and we end up with compiler invocations with a huge number of include directives, which are hard to troubleshoot. We rely on the includes
attribute because our include paths are not relative to the WORKSPACE root (https://docs.bazel.build/versions/master/bazel-and-cpp.html#include-paths), which is understandable since the project originally did not support Bazel as a build system. Since we should come up with a cleaner solution if possible.
Metadata
Metadata
Assignees
Labels
No labels