Skip to content

Commit

Permalink
#2214: Make sure to include configure_options.cmake after gathering i…
Browse files Browse the repository at this point in the history
…nformation about TPLs
  • Loading branch information
JacobDomagala committed Nov 17, 2023
1 parent 8795ad9 commit 032e528
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 26 deletions.
5 changes: 4 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -62,10 +62,13 @@ include(cmake/link_vt.cmake)
# Load packages that are required for core VT build
include(cmake/load_packages.cmake)

# Some options use TPL information (whether Perl was found etc.)
include(cmake/configure_options.cmake)

include(cmake/check_compiler.cmake)
include(cmake/nvcc_no_deprecated_gpu_targets.cmake)
include(cmake/load_bundled_libraries.cmake)
include(cmake/configure_options.cmake)


# Primary VT build
add_subdirectory(src)
Expand Down
4 changes: 0 additions & 4 deletions cmake/configure_options.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -84,10 +84,6 @@ define_option(vt_mpi_guards "user MPI prevention guards"
ON vt_feature_cmake_mpi_access_guards
)

if(vt_zoltan_enabled AND NOT vt_zoltan_found)
message(STATUS "vt_zoltan_enabled=ON but Zoltan wasn't found!")
set(vt_zoltan_enabled OFF)
endif()
define_option(vt_zoltan_enabled "Zoltan" "Build VT with Zoltan" OFF vt_feature_cmake_zoltan)

if(NOT LOWERCASE_CMAKE_BUILD_TYPE STREQUAL "release")
Expand Down
3 changes: 0 additions & 3 deletions cmake/load_packages.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@ include(cmake/load_mpi_package.cmake)
# ZLIB package
include(cmake/load_zlib_package.cmake)

# Discover and load threading configuration
# include(cmake/load_threading_package.cmake)

# Perl is used to build the PMPI wrappers
find_package(Perl)

Expand Down
18 changes: 0 additions & 18 deletions cmake/load_threading_package.cmake

This file was deleted.

0 comments on commit 032e528

Please sign in to comment.