feat: add dbus-activatable desktop entry for dde-shell#1600
Conversation
1. Add a new desktop entry file org.deepin.dde-shell.desktop in the misc directory 2. Configure the entry as a DBusActivatable application with Exec and TryExec set to /bin/false 3. Set NoDisplay to true to hide from menus and launchers 4. Categorize it under System for organization purposes 5. This enables the DDE shell service to be started on demand via D- Bus activation Log: Added D-Bus activation support for DDE Shell service Influence: 1. Verify that dde-shell can be activated via D-Bus when needed 2. Test that the entry does not appear in application menus or launchers 3. Confirm that the service starts on demand and stops when no longer needed feat: 为 dde-shell 添加 D-Bus 激活的 desktop 条目 1. 在 misc 目录中添加新的 desktop 文件 org.deepin.dde-shell.desktop 2. 配置该条目为 DBusActivatable 应用,Exec 和 TryExec 设置为 /bin/false 3. 设置 NoDisplay 为 true 以隐藏于菜单和启动器中 4. 归类到 System 类别以便于组织管理 5. 这使得 DDE shell 服务可以通过 D-Bus 激活按需启动 Log: 新增 DDE Shell 服务的 D-Bus 激活支持 Influence: 1. 验证 dde-shell 能否在需要时通过 D-Bus 激活 2. 测试该条目不会出现在应用菜单或启动器中 3. 确认服务按需启动并在不需要时自动停止
There was a problem hiding this comment.
Sorry @18202781743, you have reached your weekly rate limit of 500000 diff characters.
Please try again later or upgrade to continue using Sourcery
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 18202781743 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
deepin pr auto review你好!我是CodeGeeX。我已仔细审查了你提供的Git Diff。本次修改的主要目的是为 整体来看,修改方向是正确的,但在代码安全、语法逻辑和代码质量方面有一些值得注意和改进的地方。以下是详细的审查意见: 1. 代码安全
2. 语法逻辑🔍 Debian Install 文件的通配符逻辑
🔍 SPDX 版权年份的更新逻辑
3. 代码质量💡
💡
4. 代码性能
综合修改建议如果采纳上述建议,你的代码可以修改为如下形式: misc/org.deepin.dde-shell.desktop [Desktop Entry]
Type=Application
Name=Deepin Desktop Shell
Name[zh_CN]=深度桌面壳
Comment=Deepin desktop shell service
Comment[zh_CN]=深度桌面壳服务
Icon=dde-shell
Exec=/usr/bin/dde-shell
TryExec=/usr/bin/dde-shell
DBusActivatable=true
NoDisplay=true
Terminal=false
StartupNotify=false
Categories=System;debian/dde-shell.install shell/CMakeLists.txt & shell/main.cpp |
| Type=Application | ||
| Name=Deepin Desktop Shell | ||
| Comment=Deepin desktop shell service | ||
| Icon=dde-shell |
misc directory
TryExec set to /bin/false
Bus activation
Log: Added D-Bus activation support for DDE Shell service
Influence:
needed
feat: 为 dde-shell 添加 D-Bus 激活的 desktop 条目
Log: 新增 DDE Shell 服务的 D-Bus 激活支持
Influence: