Skip to content
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

Downgrade gcc to 4.8 #18614

Merged
merged 12 commits into from
Jul 18, 2019
Merged

Conversation

JiabinYang
Copy link
Contributor

@JiabinYang JiabinYang commented Jul 12, 2019

This PR will downgrade gcc to 4.8 for latest-dev, we test it on these case:
image

And this downgrade will cause link error on Debug mode which I fixed in PR #18558
Some question has already been answered in PR #18462

Dockerfile Outdated
@@ -166,4 +195,4 @@ RUN mkdir /var/run/sshd
RUN echo 'root:root' | chpasswd
RUN sed -ri 's/^PermitRootLogin\s+.*/PermitRootLogin yes/' /etc/ssh/sshd_config
RUN sed -ri 's/UsePAM yes/#UsePAM yes/g' /etc/ssh/sshd_config
EXPOSE 22
EXPOSE 22
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A small format issue, add end of line at last line. Other things LGTM

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, thx

Copy link
Contributor

@luotao1 luotao1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with comments in #18462

zhhsplendid
zhhsplendid previously approved these changes Jul 12, 2019
Copy link
Member

@zhhsplendid zhhsplendid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

zhhsplendid
zhhsplendid previously approved these changes Jul 16, 2019
@JiabinYang JiabinYang merged commit 898237c into PaddlePaddle:develop Jul 18, 2019
gavin1332 pushed a commit to gavin1332/Paddle that referenced this pull request Jul 18, 2019
* test=develop, fix docker with paddle nccl problem

* test=develop, downgrade gcc to 4.8 for latest-dev

* test=develop, downgrade gcc to 4.8 for latest-dev

* test=develop, modify cmake to renew all third_party

* test=develop, invoke ci

* test=develop, invoke ci

* test=develop, complie python with wide-unicode

* test=deveop, refine env settings

* test=deveop, refine env settings
@JiabinYang JiabinYang deleted the downgrade_gcc_to_4.8 branch January 14, 2021 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants