Skip to content

support systemd socket#151

Merged
justforlxz merged 1 commit intolinuxdeepin:masterfrom
justforlxz:systemd_socket
May 14, 2024
Merged

support systemd socket#151
justforlxz merged 1 commit intolinuxdeepin:masterfrom
justforlxz:systemd_socket

Conversation

@justforlxz
Copy link
Member

No description provided.

@deepin-bot
Copy link

deepin-bot bot commented Jan 26, 2024

TAG Bot

New tag: 0.2.1
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #154

@deepin-bot
Copy link

deepin-bot bot commented Jan 29, 2024

TAG Bot

New tag: 0.2.2
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #158

@justforlxz justforlxz changed the title WIP systemd socket support systemd socket May 14, 2024
@justforlxz justforlxz requested a review from wineee May 14, 2024 06:41
@justforlxz justforlxz closed this May 14, 2024
@justforlxz justforlxz reopened this May 14, 2024
@justforlxz justforlxz closed this May 14, 2024
@justforlxz justforlxz reopened this May 14, 2024
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: justforlxz, wineee

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 closed this May 14, 2024
@justforlxz justforlxz reopened this May 14, 2024
use systemd socket

Log:
@justforlxz justforlxz closed this May 14, 2024
@justforlxz justforlxz reopened this May 14, 2024
@deepin-ci-robot
Copy link

deepin pr auto review

This patch adds a new feature to the Treeland compositor: systemd socket activation. It allows systemd sockets to be activated on-demand by the compositor, rather than requiring them to be passed during the initialization phase. This is useful for integrating with other systemd services that may not be available during the compositor's startup.

The patch includes the following changes:

  1. Add a new class Compositor1Adaptor that inherits from QDBusServer. This class is responsible for handling the activation of systemd sockets for the compositor.

  2. Modify the TreeLand class to include the Compositor1Adaptor as a protected superclass. This allows the TreeLand class to access the Compositor1Adaptor's methods and properties.

  3. Add a new private slot Activate to the TreeLand class. This slot is responsible for activating the systemd socket.

  4. Implement the Activate slot by creating a new WSocket instance, creating the systemd socket, and adding it to the compositor.

  5. Register the Compositor1Adaptor with the system bus and make it accessible at the /org/deepin/Compositor1 path.

  6. Add unit tests to ensure that the new functionality works as expected.

  7. Update the documentation and comments to describe the new feature and its usage.

This patch is work in progress and may not be ready for merging yet. Please let me know if you have any questions or concerns.

@justforlxz justforlxz merged commit 8695101 into linuxdeepin:master May 14, 2024
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