-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Docker build #1661
Docker build #1661
Conversation
paddle/scripts/docker/build.sh
Outdated
RUN dpkg --force-all -i /usr/local/opt/paddle/deb/*.deb && rm -f /usr/local/opt/paddle/deb/*.deb | ||
ADD build/*.deb /usr/local/opt/paddle/deb/ | ||
ADD build/py_paddle*linux*.whl /usr/local/opt/paddle/share/wheels/ | ||
ADD build/paddle*.whl /usr/local/opt/paddle/share/wheels/ |
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.
这三个ADD没必要哈,因为deb文件中已经包含了paddle python的库了。下面执行paddle version会自动安装
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.
Done. Removed last two, I think the first one is necessary.
paddle/scripts/docker/build.sh
Outdated
ADD build/paddle*.whl /usr/local/opt/paddle/share/wheels/ | ||
|
||
# run paddle version to install python packages first | ||
RUN dpkg -i /usr/local/opt/paddle/deb/*.deb && rm -f /usr/local/opt/paddle/deb/*.deb && paddle version | ||
|
||
ENV PATH="/usr/local/opt/paddle/bin/:${PATH}" |
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.
这个ENV也可以去掉了paddle 会安装在/usr/bin下
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.
Done.
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
Fixes: #1655