feat: support Qt 6.10

[ComixHe]

No description provided.

[github-actions]

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

[deepin-ci-robot]

deepin pr auto review

我对这段CMake代码进行审查，提出以下改进意见：

1. 代码逻辑和语法：

• 代码整体语法正确，逻辑清晰
• 版本判断使用了STREQUAL和VERSION_GREATER_EQUAL，这是合适的做法
• 错误处理使用了FATAL_ERROR，这是适当的

2. 代码质量：

• 使用了明确的变量名（QT_VERSION_MAJOR, Qt6Core_VERSION），提高了可读性
• 使用了GLOB来收集头文件，这是CMake的常见做法

3. 性能考虑：

• GLOB操作在配置时执行一次，不会影响构建性能
• 版本检查在配置时进行，不会影响运行时性能

4. 安全性：

• 错误处理完善，对不支持的Qt版本会报错
• 变量使用明确，避免了潜在的命名冲突

改进建议：

1. 变量作用域优化：

if("${QT_VERSION_MAJOR}" STREQUAL "5")
    find_package(Qt5 REQUIRED COMPONENTS X11Extras)
elseif("${QT_VERSION_MAJOR}" STREQUAL "6")
    if (${Qt6Core_VERSION} VERSION_GREATER_EQUAL "6.10.0")
      set(QT_NO_PRIVATE_MODULE_WARNING ON)
      find_package(Qt6 REQUIRED COMPONENTS GuiPrivate WidgetsPrivate PrintSupportPrivate)
    endif()
else()
    message(FATAL_ERROR "Unsupported Qt Version: ${QT_VERSION_MAJOR}")
endif()

建议将QT_NO_PRIVATE_MODULE_WARNING的作用域限制在if块内：

if("${QT_VERSION_MAJOR}" STREQUAL "5")
    find_package(Qt5 REQUIRED COMPONENTS X11Extras)
elseif("${QT_VERSION_MAJOR}" STREQUAL "6")
    if (${Qt6Core_VERSION} VERSION_GREATER_EQUAL "6.10.0")
      set(QT_NO_PRIVATE_MODULE_WARNING ON PARENT_SCOPE)
      find_package(Qt6 REQUIRED COMPONENTS GuiPrivate WidgetsPrivate PrintSupportPrivate)
    endif()
else()
    message(FATAL_ERROR "Unsupported Qt Version: ${QT_VERSION_MAJOR}")
endif()

2. 版本检查的健壮性：
   建议添加更详细的版本检查信息：

if("${QT_VERSION_MAJOR}" STREQUAL "5")
    find_package(Qt5 REQUIRED COMPONENTS X11Extras)
    message(STATUS "Found Qt5 version: ${Qt5Core_VERSION}")
elseif("${QT_VERSION_MAJOR}" STREQUAL "6")
    if(${Qt6Core_VERSION} VERSION_LESS "6.10.0")
        message(WARNING "Qt6 version ${Qt6Core_VERSION} is below recommended version 6.10.0")
    endif()
    set(QT_NO_PRIVATE_MODULE_WARNING ON PARENT_SCOPE)
    find_package(Qt6 REQUIRED COMPONENTS GuiPrivate WidgetsPrivate PrintSupportPrivate)
    message(STATUS "Found Qt6 version: ${Qt6Core_VERSION}")
else()
    message(FATAL_ERROR "Unsupported Qt Version: ${QT_VERSION_MAJOR}. Only Qt5 and Qt6 are supported.")
endif()

3. 添加更多的错误处理：

if("${QT_VERSION_MAJOR}" STREQUAL "5")
    find_package(Qt5 REQUIRED COMPONENTS X11Extras)
    if(NOT Qt5_FOUND)
        message(FATAL_ERROR "Failed to find required Qt5 components")
    endif()
elseif("${QT_VERSION_MAJOR}" STREQUAL "6")
    if(${Qt6Core_VERSION} VERSION_GREATER_EQUAL "6.10.0")
        set(QT_NO_PRIVATE_MODULE_WARNING ON PARENT_SCOPE)
        find_package(Qt6 REQUIRED COMPONENTS GuiPrivate WidgetsPrivate PrintSupportPrivate)
        if(NOT Qt6_FOUND)
            message(FATAL_ERROR "Failed to find required Qt6 components")
        endif()
    else()
        message(WARNING "Qt6 private modules require version 6.10.0 or higher")
    endif()
else()
    message(FATAL_ERROR "Unsupported Qt Version: ${QT_VERSION_MAJOR}. Only Qt5 and Qt6 are supported.")
endif()

这些改进将使代码更加健壮、可维护，并提供更好的错误处理和反馈。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, ComixHe

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment