Delete downloaded BUILD files for makefile dependencies immediately after extracting and also at the beginning of ./configure. This will prevent bazel from erroneously parsing contrib/makefile/downloads during configuration if a makefile build has previously been performed.

Change: 145838744

2 files changed, 10 insertions, 0 deletions

diff --git a/configure b/configure
index a8e7bb7738..6cd5c2f3db 100755
--- a/configure
+++ b/configure
@@ -44,6 +44,13 @@ function bazel_clean_and_fetch() {
   bazel fetch "//tensorflow/... -//tensorflow/examples/android/..."
 }
 
+# Delete any leftover BUILD files from the Makefile build, which would interfere
+# with Bazel parsing.
+MAKEFILE_DOWNLOAD_DIR=tensorflow/contrib/makefile/downloads
+if [ -d "${MAKEFILE_DOWNLOAD_DIR}" ]; then
+  find ${MAKEFILE_DOWNLOAD_DIR} -type f -name '*BUILD' -delete
+fi
+
 ## Set up python-related environment settings
 while true; do
   fromuser=""
diff --git a/tensorflow/contrib/makefile/download_dependencies.sh b/tensorflow/contrib/makefile/download_dependencies.sh
index 7aebe18767..58d1dc6f0a 100755
--- a/tensorflow/contrib/makefile/download_dependencies.sh
+++ b/tensorflow/contrib/makefile/download_dependencies.sh
@@ -44,6 +44,9 @@ download_and_extract() {
   echo "downloading ${url}" >&2
   mkdir -p "${dir}"
   curl -Ls "${url}" | tar -C "${dir}" --strip-components=1 -xz
+
+  # Delete any potential BUILD files, which would interfere with Bazel builds.
+  find "${dir}" -type f -name '*BUILD' -delete
 }
 
 download_and_extract "${EIGEN_URL}" "${DOWNLOADS_DIR}/eigen"