sync: from linuxdeepin/dde-session-shell

[deepin-ci-robot]

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

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

[sourcery-ai]

Sorry @deepin-ci-robot, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

[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-bot]

TAG Bot

New tag: 6.0.50
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #478

[deepin-bot]

TAG Bot

New tag: 6.0.51
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #483

[deepin-ci-robot]

deepin pr auto review

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

1. 语法逻辑：

• 修改整体语法正确，没有明显的语法错误
• CMakeLists.txt 中的修改符合 CMake 语法规范
• C++ 代码中的字符串拼接使用正确

2. 代码质量：
   优点：

• 使用 CMAKE_INSTALL_LIBDIR 变量替代硬编码的 "lib" 路径，提高了代码的可维护性和可移植性
• 在主 CMakeLists.txt 中添加了编译定义，使得 C++ 代码能够使用安装路径
• 统一了所有插件的安装路径配置

改进建议：

• 建议在 CMakeLists.txt 中添加对 CMAKE_INSTALL_LIBDIR 的检查，确保变量已定义
• 可以考虑使用 GNUInstallDirs 模块来标准化安装目录

3. 代码性能：

• 修改对性能没有显著影响
• 字符串拼接操作在运行时执行，但由于是初始化阶段的一次性操作，影响很小

4. 代码安全：
   优点：

• 使用 CMake 变量而不是硬编码路径，提高了安全性
• 避免了可能的路径注入问题

改进建议：

• 建议在 CMakeLists.txt 中添加路径验证，确保 CMAKE_INSTALL_PREFIX 和 CMAKE_INSTALL_LIBDIR 是合法的路径
• 可以考虑使用 file(TO_CMAKE_PATH) 来规范化路径格式

5. 其他建议：

• 建议在代码中添加注释说明为什么需要使用 CMAKE_INSTALL_FULL_LIBDIR
• 可以考虑将路径定义放在一个单独的配置文件中，便于统一管理
• 建议添加单元测试来验证路径配置的正确性

总体来说，这是一个良好的改进，提高了代码的可维护性和可移植性。主要改进点是使用 CMake 变量替代硬编码路径，使代码更加灵活和安全。建议的改进主要集中在增加安全检查和文档说明方面。

[deepin-bot]

TAG Bot

New tag: 6.0.52
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #485