ML-Build: Bazel Version for building Tensorflow

rsnk96 / Ubuntu-Setup-Scripts

Scripts to help you set up your Ubuntu quickly, especially if you're in any subfield of Data Science or AI!

Apache License 2.0

128 stars 76 forks source link

ML-Build: Bazel Version for building Tensorflow #32

Closed rajat2004 closed 5 years ago

rajat2004 commented 5 years ago

Currently, the ML-Build script is failing Travis due to unsupported Bazel version

Specifying the older version during apt install doesn't work since the repository only contains the latest bazel version. See this discussion: https://github.com/bazelbuild/continuous-integration/issues/128

Right now, this can be fixed by manual installation of the .deb package with the exact version required by the release. But this will have to be updated as and when new Tensorflow relases occur

rsnk96 commented 5 years ago

Fixed thru PR

rajat2004 commented 5 years ago

ML-Build.sh is grabbing a different version of Tensorflow now - 1.12.1 which requires Bazel 0.24.1. Should we do something like fix the Tensorflow version depending on the most stable one and fix Bazel also for that?

Or maybe see if it's possible to parse some file in the Tensorflow repo and download the Bazel version accordingly

rajat2004 commented 5 years ago

https://github.com/tensorflow/tensorflow/blob/343ba9424baab34ea4b8c9ae6d73f93e51af9837/configure.py#L1390 This specifies the minimum Bazel version required.