Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Conversation

@lebeg
Copy link
Contributor

@lebeg lebeg commented Aug 17, 2018

… Fix perms. (#12125)"

This reverts commit 1b60478.

Description

Unfortunately this change had broken a lot of things including website build and model compatibility checker. We have tested 2 solutions: a fix on top of latest master and reverting this commit. The later did the job nicely while a fix required more work to be done.

@marcoabreu marcoabreu added the pr-awaiting-merge Review and CI is complete. Ready to Merge label Aug 17, 2018
@marcoabreu marcoabreu merged commit 2ff1837 into apache:master Aug 17, 2018
marcoabreu added a commit that referenced this pull request Aug 17, 2018
piyushghai added a commit to piyushghai/incubator-mxnet that referenced this pull request Aug 17, 2018
larroy pushed a commit to larroy/mxnet that referenced this pull request Aug 20, 2018
… cripts. Fix perms. (apache#12125)" (apache#12223)"

This reverts commit 2ff1837.

add docs_requirements
larroy pushed a commit to larroy/mxnet that referenced this pull request Aug 21, 2018
… cripts. Fix perms. (apache#12125)" (apache#12223)"

This reverts commit 2ff1837.

add docs_requirements
larroy pushed a commit to larroy/mxnet that referenced this pull request Aug 21, 2018
… cripts. Fix perms. (apache#12125)" (apache#12223)"

This reverts commit 2ff1837.

add docs_requirements
XinYao1994 pushed a commit to XinYao1994/incubator-mxnet that referenced this pull request Aug 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

pr-awaiting-merge Review and CI is complete. Ready to Merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants