From 54cfd26b6c7bcabcf6cf14eb28574f8de74b933e Mon Sep 17 00:00:00 2001 From: Garcia Ruiz Date: Tue, 20 Jul 2021 23:15:09 +0200 Subject: [PATCH] Fixed cmake Python so it correctly reinstalls a local python package if its setup.py changes Signed-off-by: Garcia Ruiz --- cmake/LYPython.cmake | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/cmake/LYPython.cmake b/cmake/LYPython.cmake index c188bac983..21e220f1d8 100644 --- a/cmake/LYPython.cmake +++ b/cmake/LYPython.cmake @@ -143,7 +143,10 @@ function(ly_pip_install_local_package_editable package_folder_path pip_package_n # we only ever need to do this once per runtime install, since its a link # not an actual install: - if(EXISTS ${stamp_file}) + # If setup.py changes we must reinstall the package in case its dependencies changed + set_property(DIRECTORY APPEND PROPERTY CMAKE_CONFIGURE_DEPENDS ${package_folder_path}/setup.py) + + if(EXISTS ${stamp_file} AND ${stamp_file} IS_NEWER_THAN ${package_folder_path}/setup.py) ly_package_is_newer_than(${LY_PYTHON_PACKAGE_NAME} ${stamp_file} package_is_newer) if (NOT package_is_newer) # no need to run the command again, as the package is older than the stamp file