Skip to content

Conversation

@zsien
Copy link
Contributor

@zsien zsien commented Nov 28, 2024

dde-desktop 已被 dde-shell 替代

Bug: https://pms.uniontech.com/bug-view-282785.html

@zsien zsien requested a review from a team November 28, 2024 08:37
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: zsien

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

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
Copy link

deepin pr auto review

代码审查意见:

  1. 代码删除

    • stopDesktop 函数和 DDE_DESKTOP_SERVICE 宏在 sessionmanager.cpputils.h 中被删除,但在 systemd/CMakeLists.txtdde-session-initialized.target.wants/dde-desktop.service 文件中仍然存在引用。这可能会导致编译错误或运行时错误。建议确认是否确实需要删除这些代码,并更新所有相关文件以反映这一变化。
  2. 代码注释

    • sessionmanager.cpp 中,stopDesktop 函数被注释掉,但没有提供替代方案或说明为什么删除。如果这个函数确实不再需要,应该在注释中说明原因。如果需要保留,应该取消注释并确保其功能正确实现。
  3. 依赖管理

    • systemd/CMakeLists.txt 中,dde-desktop.service 被从 dde-session-initialized.target.wants 中删除。如果这个服务仍然需要被启动,应该恢复其引用。如果不再需要,应该更新所有相关文档和配置文件。
  4. 服务启动顺序

    • dde-session-initialized.target.wants 中,dde-desktop.service 被删除。如果这个服务在 dde-session-initialized.target 启动时需要,应该将其添加回来。确保服务的启动顺序符合系统需求。
  5. 代码维护

    • 如果 stopDesktop 函数被删除,建议检查是否有其他地方引用了这个函数,并确保这些引用也被删除或更新。
  6. 编译和运行测试

    • 在进行任何代码删除或更改后,应该运行完整的编译和测试流程,以确保没有引入新的错误或问题。

总结:

  • 确认是否确实需要删除 stopDesktop 函数和相关服务。
  • 如果删除,确保所有相关文件和配置都已更新。
  • 如果保留,确保函数实现正确,并且所有引用都已更新。
  • 运行完整的编译和测试流程,确保没有引入新的问题。

@zsien zsien merged commit cc1578f into linuxdeepin:master Nov 28, 2024
16 of 17 checks passed
@zsien zsien deleted the zs/chore/dde-desktop branch November 28, 2024 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants