sync: from linuxdeepin/dde-session-shell

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dde-session-shell.

Source-pull-request: linuxdeepin/dde-session-shell#41

[sourcery-ai]

Sorry @deepin-ci-robot, your pull request is larger than the review limit of 150000 diff characters

[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. 语法逻辑：

   • 修改是有效的，CMake版本要求从3.16提升到了3.17，这是一个合法的版本升级。
   • 语法本身没有问题，cmake_minimum_required()的用法正确。

2. 代码质量：

   • 版本升级本身不会影响代码质量，但建议在项目中记录这个变更的原因。
   • 如果项目依赖CMake 3.17的特定新特性，应该在文档中明确说明。

3. 代码性能：

   • CMake 3.17相比3.16可能有一些性能优化，但这个版本升级本身不会显著影响构建性能。
   • 如果项目确实需要利用3.17的性能改进，应该明确指出哪些功能需要这些改进。

4. 代码安全：

   • 版本升级可能会引入新的检查和警告，这有助于提前发现潜在问题。
   • 建议确保团队成员的CMake版本都满足这个新要求，避免构建环境不一致导致的问题。

5. 其他建议：

   • 建议在项目的README或文档中更新CMake版本要求。
   • 考虑添加对更高CMake版本的兼容性说明，例如"要求CMake 3.17或更高版本"。
   • 如果这是一个团队项目，确保所有开发环境都已升级到CMake 3.17或更高版本。

总体而言，这是一个简单的版本升级，但需要确保所有依赖此构建系统的环境都满足新的版本要求。