Remove default support for Google Cloud Platform that was introduced recently.
Some users do not want or need this support and it significantly increases
their dependencies and binary sizes. After this change, users must explicitly
depend on :gcs_file_system to enable this support.
Change: 143129406
diff --git a/configure b/configure
index 95f67e5..c240125 100755
--- a/configure
+++ b/configure
@@ -55,10 +55,41 @@
done
if is_windows; then
+ TF_NEED_GCP=0
TF_NEED_HDFS=0
TF_NEED_OPENCL=0
fi
+while [ "$TF_NEED_GCP" == "" ]; do
+ read -p "Do you wish to build TensorFlow with "\
+"Google Cloud Platform support? [y/N] " INPUT
+ case $INPUT in
+ [Yy]* ) echo "Google Cloud Platform support will be enabled for "\
+"TensorFlow"; TF_NEED_GCP=1;;
+ [Nn]* ) echo "No Google Cloud Platform support will be enabled for "\
+"TensorFlow"; TF_NEED_GCP=0;;
+ "" ) echo "No Google Cloud Platform support will be enabled for "\
+"TensorFlow"; TF_NEED_GCP=0;;
+ * ) echo "Invalid selection: " $INPUT;;
+ esac
+done
+
+if [ "$TF_NEED_GCP" == "1" ]; then
+ ## Verify that libcurl header files are available.
+ # Only check Linux, since on MacOS the header files are installed with XCode.
+ if [[ $(uname -a) =~ Linux ]] && [[ ! -f "/usr/include/curl/curl.h" ]]; then
+ echo "ERROR: It appears that the development version of libcurl is not "\
+"available. Please install the libcurl3-dev package."
+ exit 1
+ fi
+
+ # Update Bazel build configuration.
+ sed -i -e "s/WITH_GCP_SUPPORT = False/WITH_GCP_SUPPORT = True/" tensorflow/core/platform/default/build_config.bzl
+else
+ # Update Bazel build configuration.
+ sed -i -e "s/WITH_GCP_SUPPORT = True/WITH_GCP_SUPPORT = False/" tensorflow/core/platform/default/build_config.bzl
+fi
+
while [ "$TF_NEED_HDFS" == "" ]; do
read -p "Do you wish to build TensorFlow with "\
"Hadoop File System support? [y/N] " INPUT
diff --git a/tensorflow/core/BUILD b/tensorflow/core/BUILD
index ddad3da..95b111f 100644
--- a/tensorflow/core/BUILD
+++ b/tensorflow/core/BUILD
@@ -1351,7 +1351,6 @@
":protos_all_cc",
"//third_party/eigen3",
"//tensorflow/core/kernels:required",
- "//tensorflow/core/platform/cloud:gcs_file_system",
] + tf_additional_lib_deps(),
alwayslink = 1,
)
diff --git a/tensorflow/core/platform/default/build_config.bzl b/tensorflow/core/platform/default/build_config.bzl
index 92d2057..abad729 100644
--- a/tensorflow/core/platform/default/build_config.bzl
+++ b/tensorflow/core/platform/default/build_config.bzl
@@ -4,6 +4,7 @@
load("@protobuf//:protobuf.bzl", "py_proto_library")
# configure may change the following lines to True
+WITH_GCP_SUPPORT = False
WITH_HDFS_SUPPORT = False
# Appends a suffix to a list of deps.
@@ -176,6 +177,8 @@
def tf_additional_lib_deps():
deps = []
+ if WITH_GCP_SUPPORT:
+ deps.append("//tensorflow/core/platform/cloud:gcs_file_system")
if WITH_HDFS_SUPPORT:
deps.append("//tensorflow/core/platform/hadoop:hadoop_file_system")
return deps
diff --git a/tensorflow/python/BUILD b/tensorflow/python/BUILD
index 70b35ea..8fc4f8c 100644
--- a/tensorflow/python/BUILD
+++ b/tensorflow/python/BUILD
@@ -2339,7 +2339,6 @@
"//tensorflow/core:lib",
"//tensorflow/core/debug",
"//tensorflow/core/distributed_runtime:server_lib",
- "//tensorflow/core/platform/cloud:gcs_file_system",
"//tensorflow/tools/tfprof/internal:print_model_analysis",
"//util/python:python_headers",
] + tf_additional_lib_deps() + tf_additional_plugin_deps(),