Skip to content

Commit

Permalink
Update TODO based on 144
Browse files Browse the repository at this point in the history
  • Loading branch information
ryanmrichard committed Dec 20, 2023
1 parent f6f416e commit c797f91
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cmake/cmaize/user_api/find_or_build_dependency.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,6 @@ function(cmaize_find_or_build_dependency _fobd_name)

message(STATUS "Attempting to fetch and build ${_fobd_name}")

# TODO: Should CMakePackageManager be refactored to match the other?
if("${_fobd_PACKAGE_MANAGER}" STREQUAL "cmake")
PackageManager(get_package
"${_fobd_pm}" _fobd_tgt "${_fobd_package_specs}" ${ARGN}
Expand Down Expand Up @@ -127,7 +126,8 @@ function(cmaize_find_or_build_dependency _fobd_name)
)
list(REMOVE_DUPLICATES _fobd_install_paths)
CMaizeTarget(SET "${_fobd_tgt}" install_path "${_fobd_install_paths}")

# TODO: PIPPackageManager should be refactored to be consistent with
# CMakePackageManager's workflow.
else()
PackageManager(install_package "${_fobd_pm}" "${_fobd_package_specs}")
PackageManager(
Expand Down

0 comments on commit c797f91

Please sign in to comment.