Replace dependency on entire proto library with just the headers for contrib.

- Linking a version of the entire proto library with every .so is wasteful.
- Causes duplicate destruction of the static initialized objects causing
non-deterministic crashes.
Change: 147220106

Author: Manjunath Kudlur

BUILD

@@ -0,0 +1,28 @@
+# Description:
+# TensorFlow is an open source software library for numerical computation using
+# data flow graphs.
+
+package(
+    default_visibility = [
+        "//tensorflow:internal",
+        "//tensorflow_models:__subpackages__",
+    ],
+)
+
+licenses(["notice"])  # Apache 2.0
+
+exports_files(["LICENSE"])
+
+load(
+    "//tensorflow:tensorflow.bzl",
+    "cc_header_only_library",
+)
+
+cc_header_only_library(
+    name = "protobuf_headers",
+    includes = ["external/protobuf/src"],
+    visibility = ["//visibility:public"],
+    deps = [
+        "@protobuf//:protobuf",
+    ],
+)

tensorflow/contrib/factorization/kernels/BUILD

@@ -14,17 +14,17 @@ cc_library(
         ":clustering_ops",
         ":masked_matmul_ops",
         ":wals_solver_ops",
-        "@protobuf//:protobuf",
+        "//:protobuf_headers",
     ],
 )
 
 cc_library(
     name = "wals_solver_ops",
     srcs = ["wals_solver_ops.cc"],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
         "//third_party/eigen3",
-        "@protobuf//:protobuf",
     ],
     alwayslink = 1,
 )
@@ -33,9 +33,9 @@ cc_library(
     name = "clustering_ops",
     srcs = ["clustering_ops.cc"],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
         "//third_party/eigen3",
-        "@protobuf//:protobuf",
     ],
     alwayslink = 1,
 )
@@ -44,9 +44,9 @@ cc_library(
     name = "masked_matmul_ops",
     srcs = ["masked_matmul_ops.cc"],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
         "//third_party/eigen3",
-        "@protobuf//:protobuf",
     ],
     alwayslink = 1,
 )

tensorflow/contrib/ffmpeg/default/BUILD

@@ -16,8 +16,8 @@ cc_library(
         "//tensorflow/contrib/ffmpeg:ffmpeg_lib.h",
     ],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
-        "@protobuf//:protobuf",
     ],
 )
 

tensorflow/contrib/input_pipeline/kernels/BUILD

@@ -11,9 +11,9 @@ cc_library(
     name = "input_pipeline_kernels",
     srcs = ["input_pipeline_kernels.cc"],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
         "//third_party/eigen3",
-        "@protobuf//:protobuf",
     ],
     alwayslink = 1,
 )

tensorflow/contrib/layers/kernels/BUILD

@@ -11,9 +11,9 @@ cc_library(
     name = "bucketization_kernel",
     srcs = ["bucketization_kernel.cc"],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
         "//third_party/eigen3",
-        "@protobuf//:protobuf",
     ],
     alwayslink = 1,
 )
@@ -22,10 +22,10 @@ cc_library(
     name = "sparse_feature_cross_kernel",
     srcs = ["sparse_feature_cross_kernel.cc"],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
         "//third_party/eigen3",
         "@farmhash_archive//:farmhash",
-        "@protobuf//:protobuf",
     ],
     alwayslink = 1,
 )

tensorflow/contrib/tensor_forest/BUILD

@@ -50,9 +50,9 @@ cc_library(
     srcs = [":custom_op_sources"],
     hdrs = [":custom_op_headers"],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
         "//third_party/eigen3",
-        "@protobuf//:protobuf",
     ],
     alwayslink = 1,
 )
@@ -191,9 +191,9 @@ cc_library(
         "kernels/tree_utils.h",
     ],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
         "//third_party/eigen3",
-        "@protobuf//:protobuf",
     ],
 )
 

tensorflow/contrib/tensor_forest/hybrid/BUILD

@@ -66,9 +66,9 @@ cc_library(
         "core/ops/utils.h",
     ],
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
         "//third_party/eigen3",
-        "@protobuf//:protobuf",
     ],
 )
 

tensorflow/python/BUILD

@@ -253,8 +253,8 @@ cc_binary(
     }),
     linkshared = 1,
     deps = [
+        "//:protobuf_headers",
         "//tensorflow/core:framework_headers_lib",
-        "@protobuf//:protobuf",
     ],
 )
 

tensorflow/tensorflow.bzl

@@ -671,7 +671,7 @@ def cc_header_only_library(name, deps=[], **kwargs):
 
 def tf_custom_op_library_additional_deps():
   return [
-      "@protobuf//:protobuf",
+      "//:protobuf_headers",
       "//third_party/eigen3",
       "//tensorflow/core:framework_headers_lib",
   ]

tensorflow/tools/pip_package/BUILD

@@ -77,6 +77,7 @@ py_binary(
 filegroup(
     name = "licenses",
     data = [
+        "//:LICENSE",
         "//third_party/eigen3:LICENSE",
         "//third_party/hadoop:LICENSE.txt",
         "@boringssl//:LICENSE",