-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
[CodeStyle][py2] remove python2 installation from Dockerfile.ubuntu
#48685
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
f8a6ba1
to
730d261
Compare
3a3ed0d
to
d62f4df
Compare
463b4d2
to
f93e76b
Compare
Dockerfile.ubuntu
@pangyoki 建议的提测流程:镜像更新一般需要做一个镜像,给qa测试这个镜像功能有没有问题。看这个pr里是删除python2.7,就需要做完镜像后,测试镜像里是否没有python2.7了,默认是否是python3.7,python3.7 paddle的编译是否有问题。 |
感谢 @zhengqiwen1997 下周会帮忙build一个镜像,并走一下提测流程。 |
感谢 @zhengqiwen1997 build 镜像,地址在:zhengqiwen/paddle:latest-dev-cuda11.2.1-cudnn8-gcc82-test 辛苦 @XieYunshen 帮忙测试下镜像:
|
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.
LGTM
PR types
Others
PR changes
Others
Describe
pip install xxx
python2
python
软链到python3.7
本 PR 完成 Python2 退场计划1中的「删除非必要的环境依赖」,将
Dockerfile.ubuntu
中安装的 Python2 移除,并将对应的 pip(即命令pip
)安装相关依赖项命令移除移除 Python2.7 后,将通过软链接的方式将
python
、pip
链接到 Python3.7,即python
、pip
、python3
、pip3
、python3.7
、pip3.7
命令都是对应 3.7 版本的 Python / pip。此外做了如下修改:
pip3.x --no-cache-dir install --upgrade pip
,以精简 Dockerfile 步数setuptools-40.6.2
,仅仅下载并安装setuptools-50.3.2
Footnotes
[CodeStyle] Legacy python code cleanup tracking issue #46837 ↩