-
Notifications
You must be signed in to change notification settings - Fork 74k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make TensorFlow build with wrapper-free MSVC CROSSTOOL #11114
Make TensorFlow build with wrapper-free MSVC CROSSTOOL #11114
Conversation
jenkins, test this please |
@jart WDYT? |
@jhseu are we ok to use a non-released protobuf? |
Jenkins, test this please. |
Hmm.. Looks like CI doesn't like my change on protobuf... |
Generally it's fine to build with a non-released protobuf, but since it was reverted, seems fine to me. |
@gunan Can you update Bazel to 0.5.2 on Windows slaves? |
All machines should now be upgraded. |
@gunan Great! http://ci.tensorflow.org/job/tf-master-win-bzl/1213/console is green again! |
With the latest Bazel release (0.5.2), we can now build C++ code on Windows without the python wrapper scripts! This gives faster and more reliable build, but gcc flags in BUILD file won't be translated.
This change removes gcc flags(mostly warning flags) that will cause an error if passed to cl.exe.
The protobuf change is to get protocolbuffers/protobuf@0b059a3, not sure if there is a requirement to use release version of protobuf.
@gunan