Skip to content

fix(xsettings): restrict service to X11 session only#102

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
mhduiy:xsettings-x11-only
Jul 10, 2026
Merged

fix(xsettings): restrict service to X11 session only#102
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
mhduiy:xsettings-x11-only

Conversation

@mhduiy

@mhduiy mhduiy commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Limit the XSettings service and plugin to run only under X11 sessions, skipping Wayland.

PMS: BUG-367635

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @mhduiy, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@mhduiy mhduiy force-pushed the xsettings-x11-only branch from 33da4f4 to 00427a2 Compare July 10, 2026 03:07
Comment thread src/plugin-qt/xsettings/misc/org.deepin.dde.XSettings1.service Outdated
1. Add ExecCondition to XSettings service to skip on Wayland sessions
2. Add session type condition to plugin JSON for X11-only loading

Log: Restrict the XSettings plugin and service to X11 session only

fix(xsettings): 限制服务仅在 X11 会话下运行

1. 为 XSettings 服务添加 ExecCondition,在 Wayland 会话时跳过启动
2. 为插件 JSON 添加 sessionType 条件,仅 X11 下加载

Log: 限制 XSettings 插件和服务仅在 X11 会话下运行
PMS: BUG-367635
@mhduiy mhduiy force-pushed the xsettings-x11-only branch from 00427a2 to f11877c Compare July 10, 2026 03:20
@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mhduiy, 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:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mhduiy

mhduiy commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

/forcemerge

@deepin-bot

deepin-bot Bot commented Jul 10, 2026

Copy link
Copy Markdown

This pr force merged! (status: blocked)

@deepin-bot deepin-bot Bot merged commit a124ee8 into linuxdeepin:master Jul 10, 2026
7 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.

3 participants