sync: from linuxdeepin/qt5integration

[deepin-ci-robot]

Synchronize source files from linuxdeepin/qt5integration.

Source-pull-request: linuxdeepin/qt5integration#295

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot

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

[github-actions]

Note

[静态代码检查]

• 敏感词检查失败, 检测到1个文件存在敏感词

详情

{
    "CMakeLists.txt": [
        {
            "line": "  HOMEPAGE_URL \"https://github.com/linuxdeepin/qt5integration\"",
            "line_number": 14,
            "rule": "S35",
            "reason": "Url link | cc21178aa0"
        }
    ]
}

[deepin-ci-robot]

deepin pr auto review

我来对这个CMakeLists.txt的变更进行审查：

1. 版本管理改进：

• 优点：将版本控制逻辑重构得更加清晰，使用DTK5选项来区分不同版本
• 建议：可以添加版本验证，确保FILE_VERSION格式正确

2. 变量命名规范：

• 优点：统一使用DTK_前缀的变量名，提高了可读性
• 建议：可以考虑将版本相关的变量集中定义，便于维护

3. 测试相关改进：

• 优点：
  • 添加了enable_testing()，这是必要的
  • 将地址消毒器(address sanitizer)限制在Debug模式下
  • 添加了add_test()来注册测试
• 建议：
  • 可以考虑添加更多的编译选项，如内存泄漏检测
  • 测试相关的配置可以封装成函数，便于复用

4. 依赖管理：

• 优点：使用DTK_NAME_SUFFIX替代VERSION_SUFFIX，命名更准确
• 建议：可以考虑添加版本兼容性检查

5. 安全性改进：

• 优点：移除了不必要的asan链接
• 建议：可以添加更多安全相关的编译选项，如栈保护

具体改进建议：

1. 版本验证：

# 验证版本格式
if(NOT FILE_VERSION MATCHES "^[0-9]+\\.[0-9]+\\.[0-9]+$")
    message(FATAL_ERROR "Invalid version format: ${FILE_VERSION}")
endif()

2. 安全编译选项：

# 添加安全编译选项
if(CMAKE_BUILD_TYPE STREQUAL "Debug")
    target_compile_options(unit-tests PRIVATE 
        -fstack-protector-strong
        -D_FORTIFY_SOURCE=2
    )
endif()

3. 测试配置封装：

function(setup_test_target target)
    if(CMAKE_BUILD_TYPE STREQUAL "Debug")
        target_compile_options(${target} PRIVATE -fsanitize=address)
        target_link_options(${target} PRIVATE -fsanitize=address)
    endif()
    
    target_compile_options(${target} PRIVATE
        -g
        -fno-access-control
        -Wall
        -fsanitize-recover=address
        -O0
    )
    
    target_link_options(${target} PRIVATE
        -fsanitize-recover=address
    )
endfunction()

4. 版本兼容性检查：

# 检查Qt版本兼容性
if(QT_VERSION_MAJOR EQUAL 6)
    if(QT_VERSION VERSION_LESS "6.2")
        message(WARNING "Qt 6.2 or higher is recommended")
    endif()
endif()

5. 添加更多测试选项：

option(ENABLE_TEST_COVERAGE "Enable test coverage" OFF)
if(ENABLE_TEST_COVERAGE AND CMAKE_BUILD_TYPE STREQUAL "Debug")
    target_compile_options(unit-tests PRIVATE --coverage)
    target_link_options(unit-tests PRIVATE --coverage)
endif()

这些改进将提高代码的可维护性、安全性和可测试性。建议在实施这些更改时，确保在所有支持的平台上都能正常工作。