sync: from linuxdeepin/qt5platform-plugins

[deepin-ci-robot]

Synchronize source files from linuxdeepin/qt5platform-plugins.

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

[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/qt5platform-plugins\"",
            "line_number": 13,
            "rule": "S35",
            "reason": "Url link | 508b5ed804"
        }
    ]
}

[github-actions]

Note

[静态代码检查]

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

详情

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

[github-actions]

Note

[静态代码检查]

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

详情

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

[github-actions]

Note

[静态代码检查]

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

详情

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

[deepin-ci-robot]

deepin pr auto review

我来对这个 CMakeLists.txt 的修改进行审查：

1. 版本控制改进：
   优点：

• 将版本控制从基于文件读取改为基于 CMake 的 project() 命令
• 添加了 DTK5 选项，更灵活地控制版本构建
• 版本号处理更加规范，分离了主版本、次版本和补丁版本

建议：

• 可以考虑添加版本号的合法性检查
• 建议在设置 DTK_VERSION 时添加缓存标记，方便外部修改

2. 构建配置改进：
   优点：

• 将 AddressSanitizer 的使用限制在 Debug 构建类型中，避免 Release 版本的性能开销
• 添加了更清晰的状态消息输出
• 测试配置更加规范，使用 add_test() 显式指定测试

建议：

• 可以考虑添加更多的构建选项控制，比如是否启用测试覆盖率
• 建议添加编译器警告级别的设置

3. 代码组织改进：
   优点：

• 删除了重复的 message() 调用
• 变量命名更加清晰（如将 DTK_FILE_VERSION 改为 FILE_VERSION）
• 条件判断逻辑更加清晰

建议：

• 可以考虑将版本相关的设置组织在一个单独的函数或宏中
• 建议添加更多的注释说明各个配置选项的作用

4. 安全性改进：
   优点：

• AddressSanitizer 的使用更加谨慎，仅在 Debug 模式下启用
• 测试配置更加规范

建议：

• 可以考虑添加更多的安全检查，比如编译器版本检查
• 建议添加对构建环境的检查

5. 性能改进：
   优点：

• Release 版本不会包含 AddressSanitizer 的开销
• 条件判断更加高效

建议：

• 可以考虑添加并行构建的支持
• 建议添加预编译头文件的支持以提高编译速度

总体来说，这次修改提高了代码的可维护性和构建的灵活性，但还可以在以下几个方面进一步改进：

1. 添加更多的错误处理和检查
2. 提供更多的配置选项
3. 改进构建性能
4. 增强跨平台支持
5. 添加更多的文档说明

[github-actions]

Note

[静态代码检查]

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

详情

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