Skip to content

Commit 703920e

Browse files
committed
Split build commands as suggested by @Youw
1 parent 91ea0d1 commit 703920e

File tree

1 file changed

+11
-13
lines changed

1 file changed

+11
-13
lines changed

tensorflow_cc/cmake/TensorflowShared.cmake

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -16,17 +16,15 @@ ExternalProject_Add(
1616
COMMAND cp "${CMAKE_CURRENT_SOURCE_DIR}/patches/v1.15.0-workspace.bzl.patch" .
1717
COMMAND git apply v1.15.0-workspace.bzl.patch
1818
CONFIGURE_COMMAND tensorflow/contrib/makefile/compile_linux_protobuf.sh
19-
# Do not fail on warnings when building nsync
20-
COMMAND sed -i "s/ -Werror//g" tensorflow/contrib/makefile/compile_nsync.sh
21-
COMMAND tensorflow/contrib/makefile/compile_nsync.sh
22-
COMMAND "${CMAKE_CURRENT_BINARY_DIR}/build_tensorflow.sh"
23-
COMMAND "${CMAKE_CURRENT_SOURCE_DIR}/cmake/copy_links.sh" .
24-
# For some reason, Bazel sometimes puts the headers into
25-
# `bazel-genfiles/genfiles` and sometimes just to `bazel-genfiles`.
26-
# So we just create and include both the directories.
27-
COMMAND mkdir -p bazel-genfiles/genfiles
28-
COMMAND touch bazel-genfiles/_placeholder.h
29-
COMMAND touch bazel-genfiles/genfiles/_placeholder.h
30-
BUILD_COMMAND ""
31-
INSTALL_COMMAND ""
19+
# Do not fail on warnings when building nsync
20+
COMMAND sed -i "s/ -Werror//g" tensorflow/contrib/makefile/compile_nsync.sh
21+
COMMAND tensorflow/contrib/makefile/compile_nsync.sh
22+
BUILD_COMMAND "${CMAKE_CURRENT_BINARY_DIR}/build_tensorflow.sh"
23+
INSTALL_COMMAND "${CMAKE_CURRENT_SOURCE_DIR}/cmake/copy_links.sh" .
24+
# For some reason, Bazel sometimes puts the headers into
25+
# `bazel-genfiles/genfiles` and sometimes just to `bazel-genfiles`.
26+
# So we just create and include both the directories.
27+
COMMAND mkdir -p bazel-genfiles/genfiles
28+
COMMAND touch bazel-genfiles/_placeholder.h
29+
COMMAND touch bazel-genfiles/genfiles/_placeholder.h
3230
)

0 commit comments

Comments
 (0)