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

Add mkldnn_version.h to pip package #14899

Merged
merged 2 commits into from
May 7, 2019
Merged

Conversation

yuxihu
Copy link
Member

@yuxihu yuxihu commented May 6, 2019

Per changes in #13668, we generate a new header file mkldnn_version.h during MXNet compilation based on the template file mkldnn_version.h.in. This dynamically generated header file needs to be added to the MXNet pip package to prevent compilation failure for building Horovod with MKLDNN enabled MXNet pip package.

@yuxihu yuxihu requested a review from szha as a code owner May 6, 2019 20:42
@yuxihu
Copy link
Member Author

yuxihu commented May 6, 2019

@szha @pengzhao-intel @TaoLv Please help review.

tools/pip/setup.py Outdated Show resolved Hide resolved
@TaoLv
Copy link
Member

TaoLv commented May 7, 2019

Thank you for the fix @yuxihu. Merging now.

@TaoLv TaoLv merged commit fdd45cf into apache:master May 7, 2019
@yuxihu yuxihu deleted the add_mkldnn_version branch May 7, 2019 16:02
access2rohit pushed a commit to access2rohit/incubator-mxnet that referenced this pull request May 14, 2019
* add mkldnn_version.h to pip package

* add headers from build/install/include folder
haohuanw pushed a commit to haohuanw/incubator-mxnet that referenced this pull request Jun 23, 2019
* add mkldnn_version.h to pip package

* add headers from build/install/include folder
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants