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

Update xgboost.cmake #116

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update xgboost.cmake #116

wants to merge 1 commit into from

Conversation

ljch2018
Copy link
Contributor

Because they change the header file definition in dmlc #114

Because they change the header file definition in [dmlc](dmlc/rabit@87143de#diff-326058be0f78c6b42de88492aea89176)
@ljch2018
Copy link
Contributor Author

ljch2018 commented Feb 27, 2019

${XGBOOST_INSTALL_DIR}/dmlc-core/include/* ${XGBOOST_INSTALL_DIR}/rabit/include/*

Header files of dmlc in rabit should not be copy to the target directory, which will overwrite the right one.

@ljch2018 ljch2018 mentioned this pull request Feb 28, 2019
@XavierWD
Copy link

XavierWD commented Mar 6, 2019

所以要怎么改呢

@ljch2018
Copy link
Contributor Author

ljch2018 commented Mar 6, 2019

所以要怎么改呢

看看 files changed

@ZhangZoffy
Copy link

@XavierWD 请问一下你最后是怎么改的?

@saxh
Copy link

saxh commented Mar 22, 2019

有没有大佬出来说一下怎么弄啊

@nerffei
Copy link

nerffei commented Mar 24, 2019

有没有大佬出来说一下怎么弄啊

他的commit里面有,照着改就可以了

@smile901122
Copy link

同遇到这样的编译错误,感谢。

@2585575866
Copy link

感谢

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.

7 participants