Unverified Commit 571cad7e authored by Nikita Titov's avatar Nikita Titov Committed by GitHub
Browse files

[R-package] mark CMake option for building R package as private (#3397)

* Update install.libs.R

* Update CMakeLists.txt
parent 164818b0
...@@ -14,7 +14,7 @@ OPTION(USE_HDFS "Enable HDFS support (EXPERIMENTAL)" OFF) ...@@ -14,7 +14,7 @@ OPTION(USE_HDFS "Enable HDFS support (EXPERIMENTAL)" OFF)
OPTION(USE_TIMETAG "Set to ON to output time costs" OFF) OPTION(USE_TIMETAG "Set to ON to output time costs" OFF)
OPTION(USE_DEBUG "Set to ON for Debug mode" OFF) OPTION(USE_DEBUG "Set to ON for Debug mode" OFF)
OPTION(BUILD_STATIC_LIB "Build static library" OFF) OPTION(BUILD_STATIC_LIB "Build static library" OFF)
OPTION(BUILD_FOR_R "Set to ON if building lib_lightgbm for use with the R package" OFF) OPTION(__BUILD_FOR_R "Set to ON if building lib_lightgbm for use with the R package" OFF)
if(APPLE) if(APPLE)
OPTION(APPLE_OUTPUT_DYLIB "Output dylib shared library" OFF) OPTION(APPLE_OUTPUT_DYLIB "Output dylib shared library" OFF)
...@@ -67,7 +67,7 @@ if(USE_SWIG) ...@@ -67,7 +67,7 @@ if(USE_SWIG)
endif() endif()
endif(USE_SWIG) endif(USE_SWIG)
if(BUILD_FOR_R) if(__BUILD_FOR_R)
list(APPEND CMAKE_MODULE_PATH "${PROJECT_SOURCE_DIR}/cmake/modules") list(APPEND CMAKE_MODULE_PATH "${PROJECT_SOURCE_DIR}/cmake/modules")
find_package(LibR REQUIRED) find_package(LibR REQUIRED)
message(STATUS "LIBR_EXECUTABLE: ${LIBR_EXECUTABLE}") message(STATUS "LIBR_EXECUTABLE: ${LIBR_EXECUTABLE}")
...@@ -75,7 +75,7 @@ if(BUILD_FOR_R) ...@@ -75,7 +75,7 @@ if(BUILD_FOR_R)
message(STATUS "LIBR_CORE_LIBRARY: ${LIBR_CORE_LIBRARY}") message(STATUS "LIBR_CORE_LIBRARY: ${LIBR_CORE_LIBRARY}")
include_directories(${LIBR_INCLUDE_DIRS}) include_directories(${LIBR_INCLUDE_DIRS})
ADD_DEFINITIONS(-DLGB_R_BUILD) ADD_DEFINITIONS(-DLGB_R_BUILD)
endif(BUILD_FOR_R) endif(__BUILD_FOR_R)
if(USE_TIMETAG) if(USE_TIMETAG)
ADD_DEFINITIONS(-DTIMETAG) ADD_DEFINITIONS(-DTIMETAG)
...@@ -235,9 +235,9 @@ list(APPEND SOURCES "src/c_api.cpp") ...@@ -235,9 +235,9 @@ list(APPEND SOURCES "src/c_api.cpp")
# Only build the R part of the library if building for # Only build the R part of the library if building for
# use with the R package # use with the R package
if(BUILD_FOR_R) if(__BUILD_FOR_R)
list(APPEND SOURCES "src/lightgbm_R.cpp") list(APPEND SOURCES "src/lightgbm_R.cpp")
endif(BUILD_FOR_R) endif(__BUILD_FOR_R)
if (BUILD_STATIC_LIB) if (BUILD_STATIC_LIB)
add_library(_lightgbm STATIC ${SOURCES}) add_library(_lightgbm STATIC ${SOURCES})
...@@ -315,13 +315,13 @@ if(WIN32 AND (MINGW OR CYGWIN)) ...@@ -315,13 +315,13 @@ if(WIN32 AND (MINGW OR CYGWIN))
TARGET_LINK_LIBRARIES(_lightgbm IPHLPAPI) TARGET_LINK_LIBRARIES(_lightgbm IPHLPAPI)
endif() endif()
if(BUILD_FOR_R) if(__BUILD_FOR_R)
if(MSVC) if(MSVC)
TARGET_LINK_LIBRARIES(_lightgbm ${LIBR_MSVC_CORE_LIBRARY}) TARGET_LINK_LIBRARIES(_lightgbm ${LIBR_MSVC_CORE_LIBRARY})
else() else()
TARGET_LINK_LIBRARIES(_lightgbm ${LIBR_CORE_LIBRARY}) TARGET_LINK_LIBRARIES(_lightgbm ${LIBR_CORE_LIBRARY})
endif(MSVC) endif(MSVC)
endif(BUILD_FOR_R) endif(__BUILD_FOR_R)
install(TARGETS lightgbm _lightgbm install(TARGETS lightgbm _lightgbm
RUNTIME DESTINATION ${CMAKE_INSTALL_PREFIX}/bin RUNTIME DESTINATION ${CMAKE_INSTALL_PREFIX}/bin
......
...@@ -170,7 +170,7 @@ if (!use_precompile) { ...@@ -170,7 +170,7 @@ if (!use_precompile) {
if (use_gpu) { if (use_gpu) {
cmake_args <- c(cmake_args, "-DUSE_GPU=ON") cmake_args <- c(cmake_args, "-DUSE_GPU=ON")
} }
cmake_args <- c(cmake_args, "-DBUILD_FOR_R=ON") cmake_args <- c(cmake_args, "-D__BUILD_FOR_R=ON")
# Pass in R version, used to help find R executable for linking # Pass in R version, used to help find R executable for linking
R_version_string <- paste( R_version_string <- paste(
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment