Skip to content
Snippets Groups Projects
Commit ef6adf49 authored by Martin Nolte's avatar Martin Nolte
Browse files

Merge branch 'feature/dont-regire-pip-II' into 'master'

Allow to skip dune_python_package_install when pip is not found II

See merge request !399

(cherry picked from commit f850031e)

f40a7649 improve DunePythonInstallPackage to handle missing pip
parent 76d7f0c9
Branches
Tags
3 merge requests!586Centralize CI config for 2.6 release,!531Update CI for 2.6 release branch,!425Merge branch 'feature/dont-regire-pip-II' into 'master'
Pipeline #
......@@ -40,11 +40,6 @@ function(dune_python_install_package)
message(WARNING "Unparsed arguments in dune_python_install_package: This often indicates typos!")
endif()
# Check for the presence of the pip package
if(NOT DUNE_PYTHON_pip_FOUND)
message(FATAL_ERROR "dune_python_install_package: Requested installations, but pip was not found!")
endif()
set(PYINST_FULLPATH ${CMAKE_CURRENT_SOURCE_DIR}/${PYINST_PATH})
if(EXISTS ${PYINST_FULLPATH}/setup.py.in)
configure_file(${PYINST_PATH}/setup.py.in ${PYINST_PATH}/setup.py)
......@@ -88,6 +83,16 @@ function(dune_python_install_package)
"${PYINST_FULLPATH}")
# Leave this function if no installation rules are required
if("${DUNE_PYTHON_INSTALL_LOCATION}" STREQUAL "none" AND NOT DUNE_PYTHON_VIRTUALENV_SETUP)
return()
endif()
# Check for the presence of the pip package
if(NOT DUNE_PYTHON_pip_FOUND)
message(FATAL_ERROR "dune_python_install_package: Requested installations, but pip was not found!")
endif()
#
# If requested, install into the configure-time Dune virtualenv
#
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment