sync: from linuxdeepin/dtkcore

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkcore.

Source-pull-request: linuxdeepin/dtkcore#530

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

[deepin-ci-robot]

deepin pr auto review

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

1. 语法逻辑改进：

• 在CMakeLists.txt中，将版本控制逻辑从基于PROJECT_VERSION_MAJOR改为使用DTK5选项更加清晰。新的实现更直观，使用option(DTK5 "Build DTK5." ON)来控制版本选择，这样更容易理解和维护。

2. 代码质量改进：

• 将版本相关的变量名从DTK_FILE_VERSION改为FILE_VERSION更加通用和合理。
• 添加了DTK_NAME_SUFFIX变量来处理版本后缀，使得代码更加灵活。
• 添加了更详细的版本信息设置，包括DTK_VERSION_MINOR和DTK_VERSION_PATCH。
• 将message()输出改为message(STATUS)，这样更符合CMake的最佳实践。

3. 代码性能改进：

• 移除了不必要的CMAKE_BUILD_TYPE检查，这个检查应该留给用户在配置时指定。
• 在tests/CMakeLists.txt中，将sanitize选项限制在Debug模式下，避免在Release模式下不必要的性能开销。

4. 代码安全改进：

• 在tests/CMakeLists.txt中添加了RPATH设置，确保动态库的正确加载。
• 修正了BUIILD_TESTING的拼写错误（应该是BUILD_TESTING）。

5. 其他建议：

• 建议在设置DTK_VERSION时添加版本验证逻辑，确保版本号的合法性。
• 可以考虑添加更多的版本兼容性检查，特别是在处理DTK5和DTK6之间的差异时。
• 在tests/CMakeLists.txt中，可以考虑添加更多的编译器警告选项，以提高代码质量。

这些修改总体上提高了代码的可维护性、可读性和可靠性，同时保持了向后兼容性。新的实现更加模块化，更容易进行版本管理和功能扩展。