Skip to content
Permalink
Browse files

FindSIP: use new sip_module_dir variable (#7370)

* 0vars.cmake, FindSIP: Update to save new sip_module_dir variable
sip4 has changed sip_mod_dir to sip_module_dir.
Store the variable for later usage

* Use only new MODULE_DIR sip variable
  • Loading branch information
LocutusOfBorg authored and 3nids committed Jul 6, 2018
1 parent 5c855b4 commit 9f050629c8e1abb8b6bd07e2d4ec9766b9aca62c
Showing with 7 additions and 7 deletions.
  1. +1 −1 cmake/FindSIP.cmake
  2. +4 −4 cmake/FindSIP.py
  3. +2 −2 mac/cmake/0vars.cmake.in
@@ -41,7 +41,7 @@ ELSE(SIP_VERSION)
STRING(REGEX REPLACE ".*\nsip_bin:([^\n]+).*$" "\\1" SIP_BINARY_PATH ${sip_config})
STRING(REGEX REPLACE ".*\ndefault_sip_dir:([^\n]+).*$" "\\1" SIP_DEFAULT_SIP_DIR ${sip_config})
STRING(REGEX REPLACE ".*\nsip_inc_dir:([^\n]+).*$" "\\1" SIP_INCLUDE_DIR ${sip_config})
STRING(REGEX REPLACE ".*\nsip_mod_dir:([^\n]+).*$" "\\1" SIP_MOD_DIR ${sip_config})
STRING(REGEX REPLACE ".*\nsip_module_dir:([^\n]+).*$" "\\1" SIP_MODULE_DIR ${sip_config})
SET(SIP_FOUND TRUE)
ENDIF(sip_config)

@@ -39,8 +39,8 @@
print("sip_bin:%s" % sipcfg.sip_bin)
print("default_sip_dir:%s" % sipcfg.default_sip_dir)
print("sip_inc_dir:%s" % sipcfg.sip_inc_dir)
if hasattr(sipcfg, "sip_mod_dir"):
print("sip_mod_dir:%s" % sipcfg.sip_mod_dir)
# SIP 4.19.10+ has new sipcfg.sip_module_dir
if hasattr(sipcfg, "sip_module_dir"):
print("sip_module_dir:%s" % sipcfg.sip_module_dir)
else:
# SIP 4.19.10+ has no sipcfg.sip_mod_dir
print("sip_mod_dir:%s" % sipcfg.sip_module_dir)
print("sip_module_dir:%s" % sipcfg.sip_mod_dir)
@@ -64,8 +64,8 @@ LIST(APPEND PYTHON_MODULE_PATHS ${_pythonpath})
IF (EXISTS "@PYQT5_MOD_DIR@")
LIST(APPEND PYTHON_MODULE_PATHS "@PYQT5_MOD_DIR@")
ENDIF ()
IF (EXISTS "@SIP_MOD_DIR@")
LIST(APPEND PYTHON_MODULE_PATHS "@SIP_MOD_DIR@")
IF (EXISTS "@SIP_MODULE_DIR@")
LIST(APPEND PYTHON_MODULE_PATHS "@SIP_MODULE_DIR@")
ENDIF ()
IF (EXISTS "@PYTHON_SITE_PACKAGES_SYS@")
LIST(APPEND PYTHON_MODULE_PATHS "@PYTHON_SITE_PACKAGES_SYS@")

0 comments on commit 9f05062

Please sign in to comment.
You can’t perform that action at this time.