sync: from linuxdeepin/dtklog

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtklog.

Source-pull-request: linuxdeepin/dtklog#24

[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/dtklog\"",
            "line_number": 13,
            "rule": "S35",
            "reason": "Url link | 6fabe13a51"
        }
    ]
}

[github-actions]

Note

[静态代码检查]

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

详情

{
    "CMakeLists.txt": [
        {
            "line": "  HOMEPAGE_URL \"https://github.com/linuxdeepin/dtklog\"",
            "line_number": 13,
            "rule": "S35",
            "reason": "Url link | 6fabe13a51"
        }
    ]
}

[deepin-ci-robot]

deepin pr auto review

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

1. 版本管理改进：

• 优点：引入了更规范的版本管理，使用DTK_VERSION_MAJOR、DTK_VERSION_MINOR和DTK_VERSION_PATCH分别管理版本号
• 优点：DTK_VERSION采用"0.x.y"格式，符合DTK库的ABI兼容性要求
• 建议：可以在文件开头添加注释说明版本号的格式规范

2. Qt版本支持优化：

• 优点：通过DTK5选项来控制Qt版本支持，逻辑更清晰
• 优点：引入DTK_NAME_SUFFIX变量，使库名命名更灵活
• 建议：可以考虑添加对Qt版本兼容性的检查，确保最低版本要求

3. 项目配置改进：

• 优点：添加了HOMEPAGE_URL，提供了项目主页信息
• 优点：使用option()替代了之前的直接设置，更符合CMake最佳实践
• 建议：可以添加更多的项目元数据，如作者信息、许可证等

4. 性能和安全相关：

• 建议：可以添加编译器警告设置，如：

if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
    add_compile_options(-Wall -Wextra -Wpedantic)
endif()

5. 代码质量改进：

• 建议：可以添加代码格式化检查，如clang-format
• 建议：可以添加静态分析工具配置，如cppcheck

6. 构建系统优化：

• 优点：改进了库的命名规则，更符合DTK的规范
• 优点：SOVERSION统一设置为"0"，确保ABI兼容性
• 建议：可以添加构建类型选项，如Debug/Release的区分

7. 文档和配置文件：

• 优点：更新了所有相关配置文件中的版本和命名
• 建议：可以在配置文件中添加更多注释，说明各配置项的用途

8. 依赖管理：

• 优点：在pc文件中明确添加了对Qt的依赖
• 建议：可以考虑添加版本检查，确保依赖的Qt版本满足最低要求

总体来说，这次修改提高了代码的可维护性和规范性，主要改进了版本管理和项目配置。建议可以考虑添加更多的构建选项和检查机制，以提高代码质量和安全性。