Commit f26801f9 authored by Mirko Bonadei's avatar Mirko Bonadei Committed by Commit Bot

Using -isystem to include Abseil headers.

Abseil requires some warning suppressions that should not be propagated
to other projects. Starting from this CL Abseil headers will be
included using -isystem instead of -I.

Bug: webrtc:8821
Change-Id: I54e0a9d4d9d7122e8fca91d61227f87160f0b373
Reviewed-on: https://chromium-review.googlesource.com/1026650
Commit-Queue: Mirko Bonadei <mbonadei@chromium.org>
Reviewed-by: default avatarNico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558230}
parent 4835a7de
......@@ -19,7 +19,18 @@ group("default") {
}
config("absl_include_config") {
include_dirs = ["."]
# Using -isystem instead of include_dirs (-I), so we don't need to suppress
# warnings coming from Abseil. Doing so would mask warnings in our own code.
if (!is_clang && is_win) {
# MSVC doesn't have -isystem, in that case we fallback to include_dirs and
# we use the warning suppression flags defined in :absl_default_cflags_cc.
include_dirs = [ "." ]
} else {
cflags = [
"-isystem",
rebase_path(".", root_build_dir),
]
}
}
config("absl_define_config") {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment