Feature - Qt 6 port #1

.github/workflows/ci.yml

@@ -343,7 +343,7 @@ jobs:
 
       - name: Install Homebrew dependencies
         run: |
-          brew install --quiet --formula cmake ninja readline sqlite3 xz zlib tcl-tk autoconf automake libtool python yasm clang-format black meson nasm pkg-config glew
+          brew install --quiet --formula cmake ninja readline sqlite3 xz zlib tcl-tk autoconf automake libtool python yasm clang-format black meson nasm pkg-config glew icu4c
 
       - name: Set QT_HOME
         if: matrix.arch-type == 'x86_64'

.gitignore

@@ -25,6 +25,14 @@ compile_commands.json
 
 cmake-*/*
 build/*
-_build/*
+_build*/*
 _install/*
 .venv/*
+
+# These are now generated by CMake using template to support Qt5 and Qt6.
+# See rvui.mu.in
+rvui.mu 
+# See window_mode.mu.in
+window_mode.mu
+# See session_manager.mu.in
+session_manager.mu

.gitmodules

@@ -3,4 +3,5 @@
 	url = https://github.com/shotgunsoftware/openrv-WFObj.git
 [submodule "src/pub"]
 	path = src/pub
-	url = https://github.com/shotgunsoftware/openrv-pub.git
+	url = [email protected]:cedrik-fuoco-adsk/openrv-pub.git
+	branch = qt6

cmake/defaults/cxx_defaults.cmake

@@ -68,7 +68,11 @@ ENDIF()
 IF(RV_VFX_PLATFORM STREQUAL CY2024)
   SET(RV_VFX_CY2024 ON)
   ADD_COMPILE_DEFINITIONS(RV_VFX_CY2024)
+  SET(RV_QT_PACKAGE_NAME "Qt6")
+  SET(RV_QT_MU_TARGET "MuQt6")
 ELSEIF(RV_VFX_PLATFORM STREQUAL CY2023)
   SET(RV_VFX_CY2023 ON)
   ADD_COMPILE_DEFINITIONS(RV_VFX_CY2023)
+  SET(RV_QT_PACKAGE_NAME "Qt5")
+  SET(RV_QT_MU_TARGET "MuQt5")
 ENDIF()

cmake/defaults/rv_options.cmake

@@ -73,6 +73,7 @@ SET(RV_VFX_PLATFORM
   "${_RV_VFX_PLATFORM}" 
   CACHE STRING "Set the VFX platform for installaing external dependencies" FORCE
 )
+
 SET_PROPERTY(
   CACHE RV_VFX_PLATFORM
   PROPERTY STRINGS ${_RV_VFX_PLATFORM}

cmake/dependencies/CMakeLists.txt

@@ -45,7 +45,15 @@ ENDIF()
 IF(RV_TARGET_WINDOWS)
   INCLUDE(expat)
 ENDIF()
-INCLUDE(qt5)
+
+# TODO_QT: Not generic enough.. But maybe we could use something like NOT CY2023
+#          since after 2023, it is Qt6
+IF(RV_VFX_PLATFORM STREQUAL CY2023)
+  INCLUDE(qt5)
+ELSEIF(RV_VFX_PLATFORM STREQUAL CY2024)
+  INCLUDE(qt6)
+ENDIF()
+
 INCLUDE(openssl) # Python3 requirement
 INCLUDE(python3)
 INCLUDE(boost)
@@ -120,8 +128,21 @@ IF(RV_TARGET_WINDOWS)
 ENDIF()
 MESSAGE(STATUS "Using PNG:          ${RV_DEPS_PNG_VERSION}")
 MESSAGE(STATUS "Using Python3:      ${RV_DEPS_PYTHON3_VERSION}")
-MESSAGE(STATUS "Using PySide2:      ${RV_DEPS_PYSIDE2_VERSION}")
-MESSAGE(STATUS "Using Qt5:          ${RV_DEPS_QT5_VERSION}")
+
+RV_VFX_SET_VARIABLE(
+  _rv_deps_pyside_version
+  CY2023 "Using PySide2:      ${RV_DEPS_PYSIDE_VERSION}"
+  CY2024 "Using PySide6:      ${RV_DEPS_PYSIDE_VERSION}"
+)
+MESSAGE(STATUS "${_rv_deps_pyside_version}")
+
+RV_VFX_SET_VARIABLE(
+  _rv_deps_qt_version
+  CY2023 "Using Qt5:          ${RV_DEPS_QT5_VERSION}"
+  CY2024 "Using Qt6:          ${RV_DEPS_QT6_VERSION}"
+)
+MESSAGE(STATUS "${_rv_deps_qt_version}")
+
 MESSAGE(STATUS "Using Raw:          ${RV_DEPS_RAW_VERSION}")
 MESSAGE(STATUS "Using Tiff:         ${RV_DEPS_TIFF_VERSION}")
 MESSAGE(STATUS "Using WebP:         ${RV_DEPS_WEBP_VERSION}")

cmake/dependencies/oiio.cmake

@@ -130,8 +130,6 @@ ENDIF()
 
 LIST(APPEND _configure_options "-DLibRaw_ROOT=${RV_DEPS_RAW_ROOT_DIR}")
 
-LIST(APPEND _configure_options "-DQt5_ROOT=${RV_DEPS_QT5_LOCATION}")
-
 IF(NOT RV_TARGET_LINUX)
   LIST(APPEND _configure_options "-DWebP_ROOT=${RV_DEPS_WEBP_ROOT_DIR}")
   # Linux has a Link error related to relocation; WebP appears not built with -fPIC. Hence OIIO will build WebP itself.

cmake/dependencies/python3.cmake

@@ -4,6 +4,7 @@
 # SPDX-License-Identifier: Apache-2.0
 #
 
+
 SET(_python3_target
     "RV_DEPS_PYTHON3"
 )
@@ -12,8 +13,10 @@ SET(_opentimelineio_target
     "RV_DEPS_OPENTIMELINEIO"
 )
 
-SET(_pyside2_target
-    "RV_DEPS_PYSIDE2"
+RV_VFX_SET_VARIABLE(
+  _pyside_target
+  CY2023 "RV_DEPS_PYSIDE2"
+  CY2024 "RV_DEPS_PYSIDE6"
 )
 
 SET(PYTHON_VERSION_MAJOR
@@ -48,10 +51,9 @@ SET(_opentimelineio_version
 )
 
 RV_VFX_SET_VARIABLE(
-  _pyside2_version
+  _pyside_version
   CY2023 "5.15.10"
-  # Need 5.15.11+ to support Python 3.11.
-  CY2024 "5.15.11"
+  CY2024 "6.5.3"
 )
 
 SET(_python3_download_url
@@ -70,13 +72,16 @@ SET(_opentimelineio_git_tag
     "v${_opentimelineio_version}"
 )
 
-SET(_pyside2_archive_url
-    "https://mirrors.ocf.berkeley.edu/qt/official_releases/QtForPython/pyside2/PySide2-${_pyside2_version}-src/pyside-setup-opensource-src-${_pyside2_version}.zip"
+RV_VFX_SET_VARIABLE(
+  _pyside_archive_url
+  CY2023 "https://mirrors.ocf.berkeley.edu/qt/official_releases/QtForPython/pyside2/PySide2-${_pyside_version}-src/pyside-setup-opensource-src-${_pyside_version}.zip"
+  CY2024 "https://mirrors.ocf.berkeley.edu/qt/official_releases/QtForPython/pyside6/PySide6-${_pyside_version}-src/pyside-setup-everywhere-src-${_pyside_version}.zip"
 )
+
 RV_VFX_SET_VARIABLE(
-  _pyside2_download_hash
+  _pyside_download_hash
   CY2023 "87841aaced763b6b52e9b549e31a493f"
-  CY2024 "8f652b08c1c74f9a80a2c0f16ff2a4ca"
+  CY2024 "515d3249c6e743219ff0d7dd25b8c8d8"
 )
 
 SET(_install_dir
@@ -103,13 +108,13 @@ IF(RV_TARGET_WINDOWS)
 ENDIF()
 
 FETCHCONTENT_DECLARE(
-  ${_pyside2_target}
-  URL ${_pyside2_archive_url}
-  URL_HASH MD5=${_pyside2_download_hash}
+  ${_pyside_target}
+  URL ${_pyside_archive_url}
+  URL_HASH MD5=${_pyside_download_hash}
   SOURCE_SUBDIR "sources" # Avoids the top level CMakeLists.txt
 )
 
-FETCHCONTENT_MAKEAVAILABLE(${_pyside2_target})
+FETCHCONTENT_MAKEAVAILABLE(${_pyside_target})
 
 SET(_python3_make_command_script
     "${PROJECT_SOURCE_DIR}/src/build/make_python.py"
@@ -136,30 +141,67 @@ IF(RV_TARGET_WINDOWS)
   LIST(APPEND _python3_make_command "${PYTHON_VERSION_MAJOR}${PYTHON_VERSION_MINOR}")
 ENDIF()
 
-SET(_pyside2_make_command_script
-    "${PROJECT_SOURCE_DIR}/src/build/make_pyside.py"
-)
-SET(_pyside2_make_command
-    python3 "${_pyside2_make_command_script}"
-)
-LIST(APPEND _pyside2_make_command "--variant")
-LIST(APPEND _pyside2_make_command ${CMAKE_BUILD_TYPE})
-LIST(APPEND _pyside2_make_command "--source-dir")
-LIST(APPEND _pyside2_make_command ${rv_deps_pyside2_SOURCE_DIR})
-LIST(APPEND _pyside2_make_command "--output-dir")
-LIST(APPEND _pyside2_make_command ${_install_dir})
-LIST(APPEND _pyside2_make_command "--temp-dir")
-LIST(APPEND _pyside2_make_command ${_build_dir})
-IF(DEFINED RV_DEPS_OPENSSL_INSTALL_DIR)
-  LIST(APPEND _pyside2_make_command "--openssl-dir")
-  LIST(APPEND _pyside2_make_command ${RV_DEPS_OPENSSL_INSTALL_DIR})
+
+# TODO_QT: Maybe we could use something like NOT CY2023
+#          since after 2023, it is Qt6
+# TODO_QT: Below code could be simplified, but for now it is faster to test.
+IF(RV_VFX_PLATFORM STREQUAL CY2023)
+  SET(_pyside_make_command_script
+      "${PROJECT_SOURCE_DIR}/src/build/make_pyside.py"
+  )
+  SET(_pyside_make_command
+      python3 "${_pyside_make_command_script}"
+  )
+
+  LIST(APPEND _pyside_make_command "--variant")
+  LIST(APPEND _pyside_make_command ${CMAKE_BUILD_TYPE})
+  LIST(APPEND _pyside_make_command "--source-dir")
+  LIST(APPEND _pyside_make_command ${rv_deps_pyside2_SOURCE_DIR})
+  LIST(APPEND _pyside_make_command "--output-dir")
+  LIST(APPEND _pyside_make_command ${_install_dir})
+  LIST(APPEND _pyside_make_command "--temp-dir")
+  LIST(APPEND _pyside_make_command ${_build_dir})
+
+  IF(DEFINED RV_DEPS_OPENSSL_INSTALL_DIR)
+    LIST(APPEND _pyside_make_command "--openssl-dir")
+    LIST(APPEND _pyside_make_command ${RV_DEPS_OPENSSL_INSTALL_DIR})
+  ENDIF()
+
+  LIST(APPEND _pyside_make_command "--python-dir")
+  LIST(APPEND _pyside_make_command ${_install_dir})
+  LIST(APPEND _pyside_make_command "--qt-dir")
+  LIST(APPEND _pyside_make_command ${RV_DEPS_QT5_LOCATION})
+  LIST(APPEND _pyside_make_command "--python-version")
+  LIST(APPEND _pyside_make_command "${RV_DEPS_PYTHON_VERSION_SHORT}")
+ELSEIF(RV_VFX_PLATFORM STREQUAL CY2024)
+  SET(_pyside_make_command_script
+    "${PROJECT_SOURCE_DIR}/src/build/make_pyside6.py"
+  )
+  SET(_pyside_make_command
+    python3 "${_pyside_make_command_script}"
+  )
+
+  LIST(APPEND _pyside_make_command "--variant")
+  LIST(APPEND _pyside_make_command ${CMAKE_BUILD_TYPE})
+  LIST(APPEND _pyside_make_command "--source-dir")
+  LIST(APPEND _pyside_make_command ${rv_deps_pyside6_SOURCE_DIR})
+  LIST(APPEND _pyside_make_command "--output-dir")
+  LIST(APPEND _pyside_make_command ${_install_dir})
+  LIST(APPEND _pyside_make_command "--temp-dir")
+  LIST(APPEND _pyside_make_command ${_build_dir})
+
+  IF(DEFINED RV_DEPS_OPENSSL_INSTALL_DIR)
+    LIST(APPEND _pyside_make_command "--openssl-dir")
+    LIST(APPEND _pyside_make_command ${RV_DEPS_OPENSSL_INSTALL_DIR})
+  ENDIF()
+
+  LIST(APPEND _pyside_make_command "--python-dir")
+  LIST(APPEND _pyside_make_command ${_install_dir})
+  LIST(APPEND _pyside_make_command "--qt-dir")
+  LIST(APPEND _pyside_make_command ${RV_DEPS_QT6_LOCATION})
+  LIST(APPEND _pyside_make_command "--python-version")
+  LIST(APPEND _pyside_make_command "${RV_DEPS_PYTHON_VERSION_SHORT}")
 ENDIF()
-LIST(APPEND _pyside2_make_command "--python-dir")
-LIST(APPEND _pyside2_make_command ${_install_dir})
-LIST(APPEND _pyside2_make_command "--qt-dir")
-LIST(APPEND _pyside2_make_command ${RV_DEPS_QT5_LOCATION})
-LIST(APPEND _pyside2_make_command "--python-version")
-LIST(APPEND _pyside2_make_command "${RV_DEPS_PYTHON_VERSION_SHORT}")
 
 IF(RV_TARGET_WINDOWS)
   IF(CMAKE_BUILD_TYPE MATCHES "^Debug$")
@@ -297,21 +339,40 @@ IF(RV_TARGET_WINDOWS
   )
 ENDIF()
 
-SET(${_pyside2_target}-build-flag
-    ${_install_dir}/${_pyside2_target}-build-flag
+SET(${_pyside_target}-build-flag
+    ${_install_dir}/${_pyside_target}-build-flag
 )
 
-ADD_CUSTOM_COMMAND(
-  COMMENT "Building PySide2 using ${_pyside2_make_command_script}"
-  OUTPUT ${${_pyside2_target}-build-flag}
-  # First PySide build script on Windows which doesn't respect '--debug' option
-  COMMAND ${CMAKE_COMMAND} -E copy ${PROJECT_SOURCE_DIR}/src/build/patch_PySide2/windows_desktop.py
-          ${rv_deps_pyside2_SOURCE_DIR}/build_scripts/platforms/windows_desktop.py
-  COMMAND ${_pyside2_make_command} --prepare --build
-  COMMAND cmake -E touch ${${_pyside2_target}-build-flag}
-  DEPENDS ${_python3_target} ${_pyside2_make_command_script} ${${_python3_target}-requirements-flag}
-  USES_TERMINAL
-)
+
+# TODO_QT: Maybe we could use something like NOT CY2023
+#          since after 2023, it is Qt6
+# TODO_QT: Below code could be simplified, but for now it is faster to test.
+IF(RV_VFX_PLATFORM STREQUAL CY2023)
+  ADD_CUSTOM_COMMAND(
+    COMMENT "Building PySide2 using ${_pyside_make_command_script}"
+    OUTPUT ${${_pyside_target}-build-flag}
+    # First PySide build script on Windows which doesn't respect '--debug' option
+    COMMAND ${CMAKE_COMMAND} -E copy ${PROJECT_SOURCE_DIR}/src/build/patch_PySide2/windows_desktop.py
+            ${rv_deps_pyside2_SOURCE_DIR}/build_scripts/platforms/windows_desktop.py
+    COMMAND ${_pyside_make_command} --prepare --build
+    COMMAND cmake -E touch ${${_pyside_target}-build-flag}
+    DEPENDS ${_python3_target} ${_pyside_make_command_script} ${${_python3_target}-requirements-flag}
+    USES_TERMINAL
+  )
+
+  SET(_build_flag_depends ${${_pyside_target}-build-flag})
+ELSEIF(RV_VFX_PLATFORM STREQUAL CY2024)
+  ADD_CUSTOM_COMMAND(
+    COMMENT "Building PySide6 using ${_pyside_make_command_script}"
+    OUTPUT ${${_pyside_target}-build-flag}
+    COMMAND ${_pyside_make_command} --prepare --build
+    COMMAND cmake -E touch ${${_pyside_target}-build-flag}
+    DEPENDS ${_python3_target} ${_pyside_make_command_script} ${${_python3_target}-requirements-flag}
+    USES_TERMINAL
+  )
+
+  SET(_build_flag_depends ${${_pyside_target}-build-flag})
+ENDIF()
 
 IF(RV_TARGET_WINDOWS)
   ADD_CUSTOM_COMMAND(
@@ -320,7 +381,7 @@ IF(RV_TARGET_WINDOWS)
     COMMAND ${CMAKE_COMMAND} -E copy_directory ${_install_dir}/lib ${RV_STAGE_LIB_DIR}
     COMMAND ${CMAKE_COMMAND} -E copy_directory ${_install_dir}/include ${RV_STAGE_INCLUDE_DIR}
     COMMAND ${CMAKE_COMMAND} -E copy_directory ${_install_dir}/bin ${RV_STAGE_BIN_DIR}
-    DEPENDS ${_python3_target} ${${_pyside2_target}-build-flag} ${${_python3_target}-requirements-flag}
+    DEPENDS ${_python3_target} ${${_python3_target}-requirements-flag} ${_build_flag_depends}
   )
   ADD_CUSTOM_TARGET(
     ${_python3_target}-stage-target ALL
@@ -333,7 +394,7 @@ ELSE()
     COMMAND ${CMAKE_COMMAND} -E copy_directory ${_install_dir}/lib ${RV_STAGE_LIB_DIR}
     COMMAND ${CMAKE_COMMAND} -E copy_directory ${_install_dir}/include ${RV_STAGE_INCLUDE_DIR}
     COMMAND ${CMAKE_COMMAND} -E copy_directory ${_install_dir}/bin ${RV_STAGE_BIN_DIR}
-    DEPENDS ${_python3_target} ${${_pyside2_target}-build-flag} ${${_python3_target}-requirements-flag}
+    DEPENDS ${_python3_target} ${${_python3_target}-requirements-flag} ${_build_flag_depends}
   )
   ADD_CUSTOM_TARGET(
     ${_python3_target}-stage-target ALL
@@ -370,8 +431,8 @@ SET(RV_DEPS_PYTHON3_VERSION
     ${_python3_version}
     CACHE INTERNAL "" FORCE
 )
-SET(RV_DEPS_PYSIDE2_VERSION
-    ${_pyside2_version}
+SET(RV_DEPS_PYSIDE_VERSION
+    ${_pyside_version}
     CACHE INTERNAL "" FORCE
 )
 

cmake/dependencies/qt5.cmake

@@ -53,6 +53,12 @@ FIND_PACKAGE(
   REQUIRED
 )
 
+get_target_property(MOC_EXECUTABLE Qt5::moc IMPORTED_LOCATION)
+get_target_property(UIC_EXECUTABLE Qt5::uic IMPORTED_LOCATION)
+
+SET(QT_MOC_EXECUTABLE "${MOC_EXECUTABLE}" CACHE STRING "Qt MOC executable")
+SET(QT_UIC_EXECUTABLE "${UIC_EXECUTABLE}" CACHE STRING "Qt UIC executable")
+
 SET(_qt_copy_message
     "Copying Qt into ${RV_STAGE_ROOT_DIR}"
 )