Skip to content

Conversation

@justforlxz
Copy link
Member

use systemd service start seatd

Log:

use systemd service start seatd

Log:
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. CMakeLists.txt文件中添加了add_subdirectory(systemd),但没有检查systemd是否已经安装或可用。建议添加一个检查机制,确保systemd包存在且可用。

  2. systemd/CMakeLists.txt文件中,使用了pkg_get_variable来获取SYSTEMD_SYSTEM_UNIT_DIR,但没有处理可能出现的错误情况。建议添加错误处理逻辑,以确保在SYSTEMD_SYSTEM_UNIT_DIR未定义时能够正确处理。

  3. systemd/override.conf文件中,ExecStart被设置为空,然后又被赋值为seatd -u dde -g dde -l debug。这可能是为了覆盖默认的ExecStart,但这样的写法可能会导致配置文件解析错误。建议检查systemd的配置文件格式,确保ExecStart的设置是正确的。

  4. services/ddm.service.in文件中,添加了Requires=seatd.serviceBefore=seatd.service,但没有检查seatd.service是否已经安装或可用。建议添加一个检查机制,确保seatd.service存在且可用。

  5. SingleWaylandDisplayServer.cpp文件中,移除了m_seatd的初始化和启动代码,这可能会影响seatd服务的启动。建议确认这一改动是否符合预期,并确保seatd服务在需要时能够正确启动。

  6. SingleWaylandDisplayServer.cpp文件中,m_seatdQProcess对象被创建但未启动。如果seatd服务需要在SingleWaylandDisplayServer对象创建时启动,建议在构造函数中调用m_seatd->start()

  7. SingleWaylandDisplayServer.cpp文件中,m_seatd的输出和错误处理使用了Lambda表达式,这可能会导致内存泄漏。建议使用std::functionQObject::connectQt::DirectConnection标志来避免内存泄漏。

  8. SingleWaylandDisplayServer.cpp文件中,m_seatd的输出和错误处理使用了readAllStandardOutput()readAllStandardError(),这可能会导致阻塞。建议使用readAll()readLine(),并确保在适当的时候调用waitForReadyRead()来避免阻塞。

  9. SingleWaylandDisplayServer.cpp文件中,m_seatd的输出和错误处理使用了qInfo()qWarning(),这可能会导致日志输出混乱。建议使用统一的日志系统,并确保日志级别和格式的一致性。

  10. SingleWaylandDisplayServer.cpp文件中,m_seatd的输出和错误处理使用了[m_seatd]捕获列表,这可能会导致m_seatd的引用计数增加。建议使用std::bindstd::function来避免引用计数问题。

以上是针对代码提交的审查意见,希望能够对您有所帮助。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, Groveer, justforlxz

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

@justforlxz justforlxz merged commit b891d5f into linuxdeepin:master Dec 5, 2024
9 checks passed
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.

4 participants