sync: from linuxdeepin/qt5platform-plugins

[deepin-ci-robot]

Synchronize source files from linuxdeepin/qt5platform-plugins.

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

[github-actions]

Note

[静态代码检查]

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

详情

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

[github-actions]

Note

[静态代码检查]

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

详情

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

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 在.github/workflows/call-auto-release.yml文件中，workflow_dispatch和workflow_call的inputs部分重复了，可以考虑合并以简化代码。

2. 在CMakeLists.txt文件中，读取VERSION文件并设置DTK_VERSION的方式是正确的，但是没有对VERSION文件不存在或内容格式错误的情况进行处理，建议添加错误处理逻辑。

3. 在VERSION.in文件中，@version@是一个占位符，应该确保在构建过程中被正确替换为实际的版本号。

4. 在linglong.yaml文件被删除后，.syncexclude文件中对应的排除规则也应该更新，以避免同步时遗漏新的版本文件。

5. 在xcb/dnotitlebarwindowhelper.cpp文件中，updateWindowEffectFromProperty和updateWindowStartUpEffectFromProperty函数中，使用v.isValid()来检查QVariant的有效性是一个好的做法，可以避免潜在的运行时错误。

6. 在.reuse/dep5文件中，Files列表中新增了VERSION文件，但是没有对应的License声明，建议补充完整。

7. 在CMakeLists.txt文件中，set(DTK_VERSION "${DTK_FILE_VERSION}" CACHE STRING "define project version")这行代码中，DTK_FILE_VERSION变量在读取文件内容后应该进行验证，确保其格式正确，避免设置错误的版本号。

8. 在VERSION文件中，版本号6.0.36的格式是正确的，但是建议检查是否有对应的版本号管理策略，确保版本号的唯一性和正确性。

9. 在CMakeLists.txt文件中，file(READ "${CMAKE_CURRENT_SOURCE_DIR}/VERSION" DTK_FILE_VERSION)这行代码读取了VERSION文件，但是没有对文件读取失败的情况进行处理，建议添加错误处理逻辑。

10. 在CMakeLists.txt文件中，string(STRIP "${DTK_FILE_VERSION}" DTK_FILE_VERSION)这行代码用于去除DTK_FILE_VERSION字符串两端的空白字符，这是一个好的做法，可以避免潜在的空格问题。

总体来说，代码改动看起来是合理的，但是需要关注文件读取和版本号管理的细节，确保代码的健壮性和正确性。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, 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": 14,
            "rule": "S35",
            "reason": "Url link | 508b5ed804"
        }
    ]
}