sync: from linuxdeepin/dtkcore

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkcore.

Source-pull-request: linuxdeepin/dtkcore#529

[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

我来对这个 Git diff 进行详细的代码审查：

1. 代码逻辑改进：

• 将版本判断逻辑从基于 PROJECT_VERSION_MAJOR 改为使用 DTK5 option，这样更清晰且易于维护
• 统一使用 DTK_NAME_SUFFIX 替代 DTK_VERSION_MAJOR，使得命名更规范
• 修复了拼写错误：BUIILD_TESTING -> BUILD_TESTING

2. 代码质量提升：

• 移除了无用的注释代码（如 add_sub_dir 宏中的注释）
• 使用 message(STATUS) 替代普通 message，提供更好的构建日志
• 改进了条件判断的可读性，如 if(NOT DTK5) 比 if(NOT DTK_VERSION_MAJOR) 更直观

3. 性能优化：

• 将 sanitizer 选项限制在 Debug 构建类型中，避免在 Release 构建中引入不必要的性能开销
• 添加了 RPATH 设置，确保测试程序能正确找到依赖库

4. 安全性改进：

• 在 tests/CMakeLists.txt 中添加了条件编译，确保 sanitizer 只在需要时启用
• 改进了版本控制逻辑，避免了版本混乱的可能性

5. 其他建议：

• 考虑在 CMakeLists.txt 顶部添加更多注释，说明 DTK5/DTK6 的区别
• 可以考虑将一些重复的配置提取为公共变量或函数
• 建议在文档生成部分添加版本检查，确保文档版本与实际版本一致

6. 文件组织：

• 配置文件和模板文件的修改保持了一致性
• 示例代码和工具的修改都遵循了新的命名规范

总体来说，这次改动提高了代码的可维护性和一致性，使得 DTK5 和 DTK6 的版本管理更加清晰。改动是正向的，建议合并。

[github-actions]

Note

[静态代码检查]

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

详情

{
    "CMakeLists.txt": [
        {
            "line": "  HOMEPAGE_URL \"https://github.com/linuxdeepin/dtkcore\"",
            "line_number": 9,
            "rule": "S35",
            "reason": "Url link | e18a6b5679"
        }
    ],
    "docs/CMakeLists.txt": [
        {
            "line": "execute_process(COMMAND git clone https://github.com/linuxdeepin/doxygen-theme.git --depth=1",
            "line_number": 41,
            "rule": "S35",
            "reason": "Url link | d2deace9b0"
        }
    ]
}