sl1pkn07 edited a comment on issue #14692: Fix cmake installation failed (Fix #13578) URL: https://github.com/apache/incubator-mxnet/pull/14692#issuecomment-485555512 super ugly quick workground ~~~ # Not install dmlc-core docs in / sed '/doc/s/^/#/g' \ -i 3rdparty/dmlc-core/CMakeLists.txt \ -i 3rdparty/tvm/3rdparty/dmlc-core/CMakeLists.txt ~~~ but this sould be fixed in upstream see https://github.com/dmlc/dmlc-core/blob/master/CMakeLists.txt#L216 the https://github.com/dmlc/dmlc-core/blob/master/CMakeLists.txt#L215 also sould be fixed, but is less problematic because is installed in the "right" path (`$prefix/./include` = `$prefix/include`) the latest one seems redundant because https://github.com/dmlc/dmlc-core/blob/master/CMakeLists.txt#L195-L201, but the option `INSTALL_INCLUDE_DIR` seems is not exposed in cmake-gui in resume: https://github.com/dmlc/dmlc-core/blob/master/CMakeLists.txt#L216 should be configurable by option or use `$(INSTALL_DATADIR)` for install in `$prefix/shared/doc` https://github.com/dmlc/dmlc-core/blob/master/CMakeLists.txt#L215 should be remove, move https://github.com/dmlc/dmlc-core/blob/master/CMakeLists.txt#L204 before https://github.com/dmlc/dmlc-core/blob/master/CMakeLists.txt#L195-L201 and simplify it like `install(DIRECTORY ${PROJECT_SOURCE_DIR}/include DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})` ive try to make a PR with this changes in the dmlc-core, but idk when is merged/reviewed greetings
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
