diff options
author | Jason Zaman <jasonzaman@gmail.com> | 2018-06-01 11:52:17 +0800 |
---|---|---|
committer | Gunhan Gulsoy <gunan@google.com> | 2018-05-31 20:52:17 -0700 |
commit | 1039ff9ee8c8c7ed09f9bb106131a50285866dd4 (patch) | |
tree | 2632563535c3867b74934ef159916f3be7a9ba87 /tensorflow/BUILD | |
parent | ae3456402ca15309a2fcb85adbaa8b464ca2d065 (diff) |
BUILD: dont force stripping (#19599)
* BUILD: dont force stripping
Build systems must not strip binaries, it makes it impossible for
distros to ship debugging symbols for packages.
bazel build has a --strip option to allow the user to generate stripped
binaries in a configurable way, that should be used instead.
https://fedoraproject.org/wiki/Packaging:Debuginfo
https://wiki.gentoo.org/wiki/Project:Quality_Assurance/Backtraces#Stripping
Signed-off-by: Jason Zaman <jason@perfinion.com>
* configure: add --strip=always to bazelrc
Diffstat (limited to 'tensorflow/BUILD')
-rw-r--r-- | tensorflow/BUILD | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/tensorflow/BUILD b/tensorflow/BUILD index f2ad16fa04..f4351f9dce 100644 --- a/tensorflow/BUILD +++ b/tensorflow/BUILD @@ -471,7 +471,7 @@ tf_cc_shared_object( # excludes all but a subset of function names. # On MacOS, the linker does not support version_script, but has an # an "-exported_symbols_list" command. -z defs disallows undefined -# symbols in object files and -s strips the output. +# symbols in object files. tf_cc_shared_object( name = "libtensorflow.so", @@ -485,7 +485,6 @@ tf_cc_shared_object( "//tensorflow:windows_msvc": [], "//conditions:default": [ "-z defs", - "-s", "-Wl,--version-script", # This line must be directly followed by the version_script.lds file "$(location //tensorflow/c:version_script.lds)", ], @@ -511,7 +510,6 @@ tf_cc_shared_object( "//tensorflow:windows_msvc": [], "//conditions:default": [ "-z defs", - "-s", "-Wl,--version-script", # This line must be directly followed by the version_script.lds file "$(location //tensorflow:tf_version_script.lds)", ], |