sync: from linuxdeepin/qt5platform-plugins

[deepin-ci-robot]

Synchronize source files from linuxdeepin/qt5platform-plugins.

Source-pull-request: linuxdeepin/qt5platform-plugins#320

[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

[deepin-ci-robot]

deepin pr auto review

我来对这个CMakeLists.txt的diff进行审查，并提出改进建议：

1. 版本管理改进：

• 原代码中直接读取VERSION文件内容作为版本号，新代码改为解析版本号的各个部分并组合，这样更规范
• 建议添加版本号验证，确保VERSION文件内容格式正确

2. 构建选项改进：

• 新增DTK5选项来控制构建DTK5还是DTK6，这是一个好的改进
• 建议：添加对DTK5选项值的验证，确保只能是ON/OFF

3. 编译选项改进：

• 将-fsanitize=address选项限制在Debug模式下是合理的改进
• 建议：考虑添加其他有用的调试选项，如：
  • -fsanitize=undefined
  • -fno-omit-frame-pointer
  • -g3（更详细的调试信息）

4. 测试相关改进：

• 添加了add_test命令，这是好的改进
• 建议：考虑添加测试超时设置

set_tests_properties(${PROJECT_NAME} PROPERTIES TIMEOUT 300)

5. 代码安全建议：

• 建议添加编译器安全选项：

if(CMAKE_BUILD_TYPE STREQUAL "Debug")
    target_compile_options(${PROJECT_NAME} PRIVATE
        -fno-access-control
        -fsanitize=address
        -fsanitize=undefined
        -fno-omit-frame-pointer
        -g3
    )
    target_link_options(${PROJECT_NAME} PRIVATE
        -fsanitize=address
        -fsanitize=undefined
    )
endif()

6. 代码质量建议：

• 建议添加更详细的构建状态输出：

message(STATUS "Build type: ${CMAKE_BUILD_TYPE}")
message(STATUS "Compiler: ${CMAKE_CXX_COMPILER_ID}")
message(STATUS "Install prefix: ${CMAKE_INSTALL_PREFIX}")

7. 性能相关建议：

• 建议添加Release模式下的优化选项：

if(CMAKE_BUILD_TYPE STREQUAL "Release")
    target_compile_options(${PROJECT_NAME} PRIVATE -O3)
endif()

8. 其他建议：

• 建议添加对编译器版本的检查，确保使用的编译器版本支持所需特性
• 考虑添加静态分析工具支持，如clang-tidy
• 建议添加警告选项：

target_compile_options(${PROJECT_NAME} PRIVATE -Wall -Wextra)

这些改进可以提高代码的安全性、可维护性和性能。

[github-actions]

Note

[静态代码检查]

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

详情

{
    "CMakeLists.txt": [
        {
            "line": "    HOMEPAGE_URL \"https://github.com/linuxdeepin/qt5platform-plugins\"",
            "line_number": 13,
            "rule": "S35",
            "reason": "Url link | 508b5ed804"
        }
    ]
}