Corrects mistake naming the override commands (#5140)
* corrects mistake naming the override commands Signed-off-by: Esteban Papp <81431996+amznestebanpapp@users.noreply.github.com> * fixes PR comments Signed-off-by: Esteban Papp <81431996+amznestebanpapp@users.noreply.github.com>
This commit is contained in:
@@ -124,9 +124,9 @@ function(ly_setup_target OUTPUT_CONFIGURED_TARGET ALIAS_TARGET_NAME absolute_tar
|
||||
cmake_path(RELATIVE_PATH target_library_output_directory BASE_DIRECTORY ${CMAKE_LIBRARY_OUTPUT_DIRECTORY} OUTPUT_VARIABLE target_library_output_subdirectory)
|
||||
endif()
|
||||
|
||||
if(COMMAND ly_install_target_override)
|
||||
if(COMMAND ly_setup_target_install_targets_override)
|
||||
# Mac needs special handling because of a cmake issue
|
||||
ly_install_target_override(TARGET ${TARGET_NAME}
|
||||
ly_setup_target_install_targets_override(TARGET ${TARGET_NAME}
|
||||
ARCHIVE_DIR ${archive_output_directory}
|
||||
LIBRARY_DIR ${library_output_directory}
|
||||
RUNTIME_DIR ${runtime_output_directory}
|
||||
|
||||
@@ -41,8 +41,8 @@ file(GENERATE
|
||||
configure_file(${LY_ROOT_FOLDER}/cmake/Platform/Mac/PreInstallSteps_mac.cmake.in ${CMAKE_BINARY_DIR}/runtime_install/PreInstallSteps_mac.cmake @ONLY)
|
||||
ly_install(SCRIPT ${CMAKE_BINARY_DIR}/runtime_install/PreInstallSteps_mac.cmake COMPONENT ${CMAKE_INSTALL_DEFAULT_COMPONENT_NAME})
|
||||
|
||||
#! ly_setup_runtime_dependencies_copy_function_override: Mac specific target installation
|
||||
function(ly_setup_runtime_dependencies_copy_function_override)
|
||||
#! ly_setup_target_install_targets_override: Mac specific target installation
|
||||
function(ly_setup_target_install_targets_override)
|
||||
|
||||
set(options)
|
||||
set(oneValueArgs TARGET ARCHIVE_DIR LIBRARY_DIR RUNTIME_DIR LIBRARY_SUBDIR RUNTIME_SUBDIR)
|
||||
@@ -109,8 +109,8 @@ function(ly_setup_runtime_dependencies_copy_function_override)
|
||||
endif()
|
||||
endfunction()
|
||||
|
||||
#! ly_install_code_function_override: Mac specific copy function to handle frameworks
|
||||
function(ly_install_code_function_override)
|
||||
#! ly_setup_runtime_dependencies_copy_function_override: Mac specific copy function to handle frameworks
|
||||
function(ly_setup_runtime_dependencies_copy_function_override)
|
||||
|
||||
configure_file(${LY_ROOT_FOLDER}/cmake/Platform/Mac/InstallUtils_mac.cmake.in ${CMAKE_BINARY_DIR}/runtime_install/InstallUtils_mac.cmake @ONLY)
|
||||
ly_install_run_script(${CMAKE_BINARY_DIR}/runtime_install/InstallUtils_mac.cmake)
|
||||
|
||||
Reference in New Issue
Block a user