Skip to content

Commit

Permalink
Remove attempts to set FORCE_BUILD_JULIA_BINDINGS.
Browse files Browse the repository at this point in the history
  • Loading branch information
rcurtin committed Dec 8, 2019
1 parent c05dc9f commit c1ba28d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 14 deletions.
10 changes: 1 addition & 9 deletions CMakeLists.txt
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ option(DISABLE_DOWNLOADS "Disable downloads of dependencies during build." OFF)
option(DOWNLOAD_ENSMALLEN "If ensmallen is not found, download it." ON) option(DOWNLOAD_ENSMALLEN "If ensmallen is not found, download it." ON)
option(DOWNLOAD_STB_IMAGE "Download stb_image for image loading." ON) option(DOWNLOAD_STB_IMAGE "Download stb_image for image loading." ON)
option(BUILD_PYTHON_BINDINGS "Build Python bindings." ON) option(BUILD_PYTHON_BINDINGS "Build Python bindings." ON)
option(BUILD_JULIA_BINDINGS "Build Julia bindings." ON)


if (WIN32) if (WIN32)
option(BUILD_SHARED_LIBS option(BUILD_SHARED_LIBS
Expand All @@ -29,15 +30,6 @@ else ()
"Compile shared libraries (if OFF, static libraries are compiled)." ON) "Compile shared libraries (if OFF, static libraries are compiled)." ON)
endif() endif()


# Detect whether the user passed BUILD_JULIA_BINDINGS in order to determine if
# we should fail if Julia isn't found.
if (BUILD_JULIA_BINDINGS AND NOT DEFINED FORCE_BUILD_JULIA_BINDINGS)
set(FORCE_BUILD_JULIA_BINDINGS ON CACHE BOOL "Force building Julia bindings.")
else()
set(FORCE_BUILD_JULIA_BINDINGS OFF CACHE BOOL "Force building Julia binidngs.")
endif()
option(BUILD_JULIA_BINDINGS "Build Julia bindings." ON)

# Build Markdown bindings for documentation. This is used as part of website # Build Markdown bindings for documentation. This is used as part of website
# generation. # generation.
option(BUILD_MARKDOWN_BINDINGS "Build Markdown bindings for website documentation." OFF) option(BUILD_MARKDOWN_BINDINGS "Build Markdown bindings for website documentation." OFF)
Expand Down
6 changes: 1 addition & 5 deletions src/mlpack/bindings/julia/CMakeLists.txt
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -2,11 +2,7 @@ if (BUILD_JULIA_BINDINGS)
## We need to check here if Julia is even available. Although actually ## We need to check here if Julia is even available. Although actually
## technically, I'm not sure if we even need to know! For the tests though we ## technically, I'm not sure if we even need to know! For the tests though we
## do. So it's probably a good idea to check. ## do. So it's probably a good idea to check.
if (FORCE_BUILD_JULIA_BINDINGS) find_package(Julia 0.7.0)
find_package(Julia 0.7.0 REQUIRED)
else ()
find_package(Julia 0.7.0)
endif ()


if (NOT JULIA_FOUND) if (NOT JULIA_FOUND)
# We can't build anything, so define the macro to do nothing. # We can't build anything, so define the macro to do nothing.
Expand Down

0 comments on commit c1ba28d

Please sign in to comment.