sync: from linuxdeepin/dde-session-shell

[deepin-ci-robot]

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

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

[sourcery-ai]

Sorry @deepin-ci-robot, your pull request is too large to review

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot, yixinshark

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

[yixinshark]

/forcemerge

[deepin-bot]

This pr force merged! (status: unstable)

[deepin-ci-robot]

deepin pr auto review

这段代码是CMake构建系统的一部分，用于管理插件目录下的子项目。我来分析一下这段代码：

1. 语法逻辑：

• 代码使用了CMake的条件判断语法 if()，这是正确的
• 使用 add_subdirectory() 来添加子项目，语法也是正确的
• 注释使用了 # 符号，符合CMake的注释规范

2. 代码质量：

• 代码结构清晰，易于理解
• 有适当的注释，表明了某些子项目被暂时禁用
• 但是，login-gesture 子目录的添加使用了双引号，这在CMake中不是必需的，虽然不会导致错误，但建议去掉引号以保持一致性

3. 代码性能：

• 这段代码本身不会影响构建性能
• 条件判断只会在配置时执行一次，不会影响构建时的性能

4. 代码安全：

• 代码中没有明显的安全问题
• 条件变量 DISABLE_DSS_SNIPE 的使用是安全的

改进建议：

1. 去掉 add_subdirectory("login-gesture") 中的双引号，改为：

add_subdirectory(login-gesture)

2. 考虑添加注释说明为什么 login-gesture 只在 DISABLE_DSS_SNIPE 条件下添加，例如：

if (DISABLE_DSS_SNIPE)
    add_subdirectory(one-key-login)
    # 仅在禁用DSS_SNIPE时添加手势登录功能
    add_subdirectory(login-gesture)
endif()

3. 考虑为被注释掉的子目录添加更详细的注释，说明为什么它们被禁用，例如：

# 以下子目录被暂时禁用
# examples: 示例代码，暂不需要构建
# assist_login: 辅助登录功能，暂不需要构建

这些建议主要是为了提高代码的可读性和可维护性，使其他开发者更容易理解这段代码的意图和背景。