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

[trival fix] fix cmake build cpp examples option #14562

Merged
merged 1 commit into from
Apr 3, 2019
Merged

Conversation

nihui
Copy link
Contributor

@nihui nihui commented Mar 29, 2019

Description

The CMakeLists.txt in cpp-package do not honor the BUILD_CPP_EXAMPLES option in the top-most CMakeLists.txt

it may be a mistake
fix this issue

https://github.com/apache/incubator-mxnet/blob/84c2ae1c45b1b23aab342ef924a357d66b0546a6/CMakeLists.txt#L47

@nihui nihui requested a review from szha as a code owner March 29, 2019 11:54
@nihui nihui changed the title fix cmake build cpp examples option [trival fix] fix cmake build cpp examples option Mar 29, 2019
@abhinavs95
Copy link
Contributor

Thank you for the fix @nihui

@mxnet-label-bot add [Build, CMake, pr-awaiting-review]

@marcoabreu marcoabreu added Build CMake CMake related bugs/issues/improvements pr-awaiting-review PR is waiting for code review labels Mar 29, 2019
Copy link
Member

@wkcn wkcn left a comment

Choose a reason for hiding this comment

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

Thank you for the fix!
LGTM : )

@piyushghai
Copy link
Contributor

@mxnet-label-bot add [Build, CMake, pr-awaiting-merge]

@marcoabreu marcoabreu added the pr-awaiting-merge Review and CI is complete. Ready to Merge label Apr 2, 2019
@wkcn wkcn merged commit 214ce63 into apache:master Apr 3, 2019
@wkcn
Copy link
Member

wkcn commented Apr 3, 2019

The PR has been merged: )
Thank you!

ZhennanQin pushed a commit to ZhennanQin/incubator-mxnet that referenced this pull request Apr 3, 2019
nswamy pushed a commit that referenced this pull request Apr 5, 2019
haohuanw pushed a commit to haohuanw/incubator-mxnet that referenced this pull request Jun 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Build CMake CMake related bugs/issues/improvements pr-awaiting-merge Review and CI is complete. Ready to Merge pr-awaiting-review PR is waiting for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants