commit a7d8f19a7713c7866fed08e8e758664ea43fa8f1
author Mirko Bonadei <mbonadei@webrtc.org> Fri Mar 29 13:34:46 2019 +0100
committer Commit Bot <commit-bot@chromium.org> Sun Mar 31 12:58:02 2019 +0000
tree 5c376d5bfe6bf5719c0eaee7d1b7354640603b39
parent 397b5b56e0c9ff69aa28a0fe37f501c3bbda2fd6

Use 'public_deps' to depend on nacl_io [1].

Some NaCl system headers live in a special directory and the
toolchain doesn't propagate the -I compiler flag [2].

A common workaround in Chromium is to use 'public_deps' in order
to propagate //native_client_sdk/src/libraries/nacl_io:nacl_io_include_dirs
one step further in the build graph.

[1] - https://cs.chromium.org/chromium/src/native_client_sdk/src/libraries/nacl_io/
[2] - -Inative_client_sdk/src/libraries/third_party/newlib-extras

Bug: chromium:925028
Change-Id: I5145b80c2ae6969f79fcbfcf93a6b05c8a122746
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/129701
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27373}

api/BUILD.gn

diff --git a/api/BUILD.gn b/api/BUILD.gn
index c0bf0bc..0b54fcb 100644
--- a/api/BUILD.gn
+++ b/api/BUILD.gn
@@ -168,11 +168,6 @@
     "../rtc_base:rtc_base_approved",
     "../rtc_base/system:rtc_export",
   ]
-
-  if (is_nacl) {
-    # This is needed by .h files included from rtc_base.
-    deps += [ "//native_client_sdk/src/libraries/nacl_io" ]
-  }
 }
 
 rtc_source_set("scoped_refptr") {

media/BUILD.gn

diff --git a/media/BUILD.gn b/media/BUILD.gn
index d02ef89..d976971 100644
--- a/media/BUILD.gn
+++ b/media/BUILD.gn
@@ -136,10 +136,6 @@
   if (!build_with_mozilla) {
     deps += [ "../p2p" ]
   }
-
-  if (is_nacl) {
-    deps += [ "//native_client_sdk/src/libraries/nacl_io" ]
-  }
 }
 
 rtc_static_library("rtc_constants") {

p2p/BUILD.gn

diff --git a/p2p/BUILD.gn b/p2p/BUILD.gn
index ce39958..bb6aed3 100644
--- a/p2p/BUILD.gn
+++ b/p2p/BUILD.gn
@@ -108,12 +108,6 @@
     "//third_party/abseil-cpp/absl/strings",
     "//third_party/abseil-cpp/absl/types:optional",
   ]
-
-  if (build_with_chromium) {
-    if (is_nacl) {
-      deps += [ "//native_client_sdk/src/libraries/nacl_io" ]
-    }
-  }
 }
 
 if (rtc_include_tests) {

rtc_base/BUILD.gn

diff --git a/rtc_base/BUILD.gn b/rtc_base/BUILD.gn
index 3b9fac4..2cc45d5 100644
--- a/rtc_base/BUILD.gn
+++ b/rtc_base/BUILD.gn
@@ -88,6 +88,7 @@
     "//third_party/abseil-cpp/absl/memory",
     "//third_party/abseil-cpp/absl/types:optional",
   ]
+  public_deps = []  # no-presubmit-check TODO(webrtc:8603)
 
   sources = [
     "bind.h",
@@ -149,14 +150,14 @@
   }
 
   if (is_nacl) {
-    deps += [ "//native_client_sdk/src/libraries/nacl_io" ]
+    public_deps += [ "//native_client_sdk/src/libraries/nacl_io" ]  # no-presubmit-check TODO(webrtc:8603)
   }
 
   if (is_android) {
     libs = [ "log" ]
   }
 
-  public_deps = [  # no-presubmit-check TODO(webrtc:8603)
+  public_deps += [  # no-presubmit-check TODO(webrtc:8603)
     ":atomicops",
     ":criticalsection",
     ":logging",
@@ -1006,7 +1007,7 @@
   }
 
   if (is_nacl) {
-    deps += [ "//native_client_sdk/src/libraries/nacl_io" ]
+    public_deps += [ "//native_client_sdk/src/libraries/nacl_io" ]  # no-presubmit-check TODO(webrtc:8603)
     defines += [ "timezone=_timezone" ]
     sources -= [ "ifaddrs_converter.cc" ]
   }

rtc_base/memory/BUILD.gn

diff --git a/rtc_base/memory/BUILD.gn b/rtc_base/memory/BUILD.gn
index d151557..3f70fca 100644
--- a/rtc_base/memory/BUILD.gn
+++ b/rtc_base/memory/BUILD.gn
@@ -44,9 +44,6 @@
   deps = [
     "..:rtc_base",
   ]
-  if (is_nacl) {
-    deps += [ "//native_client_sdk/src/libraries/nacl_io" ]
-  }
 }
 
 rtc_source_set("unittests") {