-
Notifications
You must be signed in to change notification settings - Fork 16
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
Fix Python38 Installation Issue #55
base: PublicCI
Are you sure you want to change the base?
Conversation
https://tensorflow-ci.intel.com/view/Monitor%20These/ shows blue status for all jobs. |
./configure --enable-optimizations --with-ensurepip=install --prefix=/usr/ | ||
make altinstall | ||
rm /usr/src/Python-3.8.2.tgz | ||
rm -rf /usr/src/Python-3.8.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we want to add a link so /usr/bin/python point to this new python3.8?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does not seem necessary. What is the benefit of letting /usr/bin/python point to python3.8?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RUN update-alternatives --install /usr/bin/python3 python3 /usr/bin/python3.8 0
This step was run in the docker file, so no need to do in this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds good
|
||
cd /usr/src | ||
wget https://www.python.org/ftp/python/3.8.2/Python-3.8.2.tgz | ||
tar xzf Python-3.8.2.tgz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From Google's container tensorflow/tensorflow:devel, they're using Python 3.8.10, do we want to install the same version as they have just for consistence?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I will update to 3.8.10 and re-test for a while.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tests with 3.8.10 look fine. https://tensorflow-ci.intel.com/view/Monitor%20These/job/tensorflow-oob-linux-cpu/2296/consoleFull
Bump up bazel version to 5.1.0
Update bazel to 5.1.1
Changed Dockerfile to pull from Ubuntu 18.04 and removed third party cmake repo.
Changed Dockerfile to pull from Ubuntu 18.04 and removed third party cmake repo.
Container from Ubuntu 18.04 and cmake fix
Previous method encounters apt get issue, most likely the python38 package was removed from the repository (targeting ARM architecture).