sync: from linuxdeepin/dtkcore

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkcore.

Source-pull-request: linuxdeepin/dtkcore#526

[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

[静态代码检查]

• 敏感词检查失败, 检测到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"
        }
    ]
}

[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"
        }
    ]
}

[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"
        }
    ]
}

[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"
        }
    ]
}

[deepin-ci-robot]

deepin pr auto review

我来对这个 CMake 配置文件的变更进行详细审查：

1. 版本管理改进：

• 优点：引入了 DTK5 选项来明确区分 DTK5 和 DTK6，比之前基于 PROJECT_VERSION_MAJOR 的判断更加清晰
• 建议：可以将 DTK5 选项的默认值设为 OFF，因为 DTK6 是更新的版本，应该作为默认选择

2. 命名规范优化：

• 优点：使用 DTK_NAME_SUFFIX 替代直接使用版本号，使命名更加灵活
• 建议：在 DTK_NAME_SUFFIX 的定义处添加注释说明其用途，提高代码可读性

3. 版本号处理：

• 优点：将版本号的各个部分（MAJOR、MINOR、PATCH）分开处理，便于管理
• 建议：可以添加版本号格式验证，确保版本号符合语义化版本规范

4. 路径管理：

• 优点：统一使用 DTK_VERSION_MAJOR 来管理安装路径，避免了版本混乱
• 建议：可以考虑将路径相关的变量集中定义，便于维护

5. 配置文件生成：

• 优点：配置文件的生成更加规范，使用了统一的命名空间
• 建议：可以添加配置文件的版本号，便于后续升级时的兼容性处理

6. 条件编译：

• 优点：使用 DTK5 选项进行条件编译，逻辑更清晰
• 建议：可以将条件编译相关的代码集中管理，避免散落在各处

7. 错误处理：

• 优点：移除了之前的错误消息，改用更优雅的选项处理
• 建议：可以添加更多的输入验证，确保配置的正确性

8. 代码清理：

• 优点：删除了一些不必要的注释和冗余代码
• 建议：可以添加更多的代码注释，特别是关键配置项的说明

9. 性能优化：

• 优点：减少了重复的版本号判断
• 建议：可以将常用的变量缓存起来，避免重复计算

10. 安全性：

• 优点：使用了更严格的变量命名规范
• 建议：可以添加配置文件的安全检查，防止路径遍历等安全问题

总结：
这次改动主要优化了版本管理和命名规范，使代码更加清晰和可维护。建议在后续版本中：

1. 添加更多的配置验证
2. 完善文档注释
3. 考虑向后兼容性
4. 添加单元测试确保配置正确性
5. 考虑添加配置模板，方便其他项目使用

[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"
        }
    ]
}