不应该让所有 dtk 应用都自行修改 XDG_CURRENT_DESKTOP 环境变量

[wineee]

SUMMARY | 问题概要

目前大部分 dtk 应用会判断如果 XDG_CURRENT_DESKTOP 不是 deepin 改成 deepin,少部分判断 XDG_CURRENT_DESKTOP 为空设成 deepin

让普通应用修改 XDG_CURRENT_DESKTOP 并不合理的，应该删除，相关 https://github.com/linuxdeepin/developer-center/issues/3829

• https://github.com/linuxdeepin/dde-session-shell/blob/bee7bdfcd01fc3a040ffc2f01fa2634c00a0d19f/src/app/lightdm-deepin-greeter.cpp#L197
• https://github.com/linuxdeepin/deepin-draw/blob/15fa348519d559411f37d621c39bd0a4416ffc46/src/deepin-draw/main.cpp#L72
• https://github.com/linuxdeepin/deepin-manual/blob/c17be6fefc4c017982c2aa77d44e9d30b1761c7b/src/app/dman.cpp#L31
• https://github.com/linuxdeepin/deepin-calculator/blob/313993b0df69fd77c1686c9580916d6cb82777b0/src/main.cpp#L78
• https://github.com/linuxdeepin/deepin-camera/blob/487494f1f5dec93abe78a6068cbbcf7dadfe447f/src/main.cpp#L135
• https://github.com/linuxdeepin/deepin-album/blob/3eeb8adc43ea1c255cc1d119f1f8dd1f621282a5/src/main.cpp#L44
• https://github.com/linuxdeepin/dde-file-manager/blob/26d9be2a962afb04dcfe4301c86ed6b3dd4f625e/src/apps/dde-file-manager/main.cpp#L176
• https://github.com/linuxdeepin/dde-introduction/blob/6e97b099500db70762aaa325e15d37173cc95b3a/src/main.cpp#L30
• https://github.com/linuxdeepin/deepin-font-manager/blob/1af8c8d040543ed6c356a986b7a9a7e4bf59f6cb/deepin-font-manager/main.cpp#L41
• https://github.com/linuxdeepin/deepin-picker/blob/161ee53d47959e107f13f038658a8d5f99c4d921/src/main.cpp#L28
• https://github.com/linuxdeepin/deepin-diskmanager/blob/da67e71b833fbc83dae149eeb4a93d144910403d/application/main.cpp#L84
• https://github.com/linuxdeepin/deepin-music/blob/0f0cad33039171110e26b4088d8a2f71d9618997/src/music-player/main.cpp#L34
• https://github.com/linuxdeepin/deepin-reader/blob/6032d2c5b3c83e4fadae5f230b23937263e29f52/reader/main.cpp#L30
• https://github.com/linuxdeepin/dde-session-ui/blob/cc107c251012f56833174d49f492ca56b9495fd4/dde-warning-dialog/src/main.cpp#L28
• https://github.com/linuxdeepin/dde-file-manager/blob/26d9be2a962afb04dcfe4301c86ed6b3dd4f625e/src/apps/dde-file-dialog-x11/main.cpp#L44
• https://github.com/linuxdeepin/dde-file-manager/blob/26d9be2a962afb04dcfe4301c86ed6b3dd4f625e/src/apps/dde-file-dialog/main.cpp#L41
• https://github.com/linuxdeepin/deepin-movie-reborn/blob/6b3b48b3be7e122a477f24c8da0e11382e0db06d/src/main.cpp#L94
• https://github.com/linuxdeepin/deepin-system-monitor/blob/13bc0095191449046cda5bea4c99773a31672f1e/deepin-system-monitor-main/main.cpp#L47
• https://github.com/linuxdeepin/deepin-editor/blob/db51af7c7efd5eeeff52fae9f2255086079c2e99/src/main.cpp#L39
• https://github.com/linuxdeepin/deepin-devicemanager/blob/3e60742898afd7509fc684460214c7d2d6b2a01c/deepin-devicemanager/src/main.cpp#L41
• https://github.com/linuxdeepin/deepin-deb-installer/blob/1456691704d9f47c3b2386dc1d297d2cbbcc61f1/src/deb-installer/main.cpp#L56
• https://github.com/linuxdeepin/dde-file-manager/blob/26d9be2a962afb04dcfe4301c86ed6b3dd4f625e/src/apps/dde-file-dialog-wayland/main.cpp#L44
• https://github.com/linuxdeepin/deepin-log-viewer/blob/b874dd52d9bcaf6a34ce56dc670a1e19788ef659/application/main.cpp#L30
• https://github.com/linuxdeepin/deepin-screen-recorder/blob/8d353957e3bb5d5de816e376fd6caf856deee95e/src/main.cpp#L71
• https://github.com/linuxdeepin/deepin-compressor/blob/938b02ced514955c3a7b1c45bc12aeaf89296b1b/src/main.cpp#L28
• https://github.com/linuxdeepin/deepin-image-viewer/blob/95ab92e386af576d5da866b6ab6ad8f69f801307/src/main.cpp#L33
• https://github.com/linuxdeepin/deepin-voice-note/blob/63e93024ebeb5274abae68d2a4be5f262f42887e/src/main.cpp#L32
• https://github.com/linuxdeepin/dde-calendar/blob/e07a7b5c60c74ba3bfd7f621d1ec9c98be11b6b1/calendar-client/src/main.cpp#L26
• https://github.com/linuxdeepin/dde-session-ui/blob/cc107c251012f56833174d49f492ca56b9495fd4/reset-password-dialog/main.cpp#L19

### Tasks

[kegechen]

主要是应对两种情况:

1. 非 DDE/Deepin 环境想正常加载 dxcb 就需要这么改
2. root 权限运行应用(如文管管理员权限), lightdm启动时greeter时, 这个环境变量是空的, 这时也需要应用自行适配.

没有加载 dxcb 会影响什么? 正常不会加载 deepintheme 插件, 进而影响图标查找. 影响最多的是内置图标(builtinicon) 现象就是 会显示系统标题栏(ssd), 部分图标加载失败. 类似这样

[wineee]

是否应该在 qt5platform-plugins 中处理此问题，比如改用 D_DXCB_ENABLE 之类的变量，让 dtk 应用确保使用 dxcb，在非 DDE 下修改 XDG_CURRENT_DESKTOP 不是合理的行为

cc @zccrs @BLumia

[zccrs]

这个问题有个计划。是希望某些隐式依赖 libdeepin.so 或 libdxcb.so 的核心功能提取出来，提到dtkgui里，让应用程序主动调用，比如截图中的图标问题就是这样的，当做到这样之后，程序即便不是在deepin中运行，没有使用DDE提供的插件，也不会有明显的问题。但是一些外围功能还是会有些差异，比如窗口没有了圆角效果，而且会多了系统标题栏，这种问题我觉得可以接受，因为圆角效果、窗口标题栏可以认为是DDE所提供的功能，而不是应用程序提供的。

[kegechen]

比如查找图标换成 DIconTheme::findQIcon ?

[BLumia]

dde-session-shell 之类项目里的这种代码可以直接去掉，因为这种项目本身就不会在 dde 外的桌面环境上使用。

有个计划。是希望某些隐士依赖 libdeepin.so 或 libdxcb.so 的核心功能提取出来，提到dtkgui里，让应用程序主动调用

这个有对应 issue 跟进吗？

[wineee]

dde-session-shell 之类项目里的这种代码可以直接去掉

deepin-picker 的也可以删了，这个应用不需要 qt5platform-plugins

[wineee]

在非 DDE 下使用系统标题栏没有问题，目前 dtk 应用有必须点击两次才能拖动的问题，使用系统标题栏反而正常点 (Arch KDE 下即使应用设置了XDG_CURRENT_DESKTOP=Deepin，也在使用系统标题栏

[zccrs]

dde-session-shell 之类项目里的这种代码可以直接去掉，因为这种项目本身就不会在 dde 外的桌面环境上使用。

有个计划。是希望某些隐士依赖 libdeepin.so 或 libdxcb.so 的核心功能提取出来，提到dtkgui里，让应用程序主动调用

这个有对应 issue 跟进吗？

@kegechen

[kegechen]

为了避免查找图标相关逻辑依赖 dxcb/deepintheme/iconengine 等插件. 将查找图标主要逻辑移动到 dtkgui .

• [x] iconengine 插件中主要实现代码移动 dtkgui
• [x] 查找图标的接口 DIconTheme::findQIcon 优化, 做到可以替换 QIcon::fromTheme
• [ ] DDE 主要应用替换相关接口, 删除 main.cpp 中 修改 XDG_CURRENT_DESKTOP 的部分

最终实现 DDE 应用无需依赖环境变量 XDG_CURRENT_DESKTOP 加载 dxcb 才能正常显示图标 可以接受无圆角, 系统标题栏, 模糊等需要窗管实现的效果.

[BLumia]

补充一个相关问题：

尽管根据上面讨论，我们（绝大多数情况）是不应该在应用里做这个判断的，但上述列出的进行如此判断的列表中的判断方式有的也是有误的。XDG_CURRENT_DESKTOP 本身可以是一个列表，很多判断都假定这个环境变量的值是单一桌面环境名称了。所以如果最终确定某个位置确需进行判断的话，需要考虑此环境变量内可能是由冒号分隔的多个值的情况。

[kegechen]

QIcon::fromTheme ==> DIconTheme::findQIcon 即可实现 dtk 应用查找图标不再依赖 dde-qt5integration.

dtkgui > 5.6.12

[felixonmars]

这个问题有个计划。是希望某些隐式依赖 libdeepin.so 或 libdxcb.so 的核心功能提取出来，提到dtkgui里，让应用程序主动调用，比如截图中的图标问题就是这样的，当做到这样之后，程序即便不是在deepin中运行，没有使用DDE提供的插件，也不会有明显的问题。但是一些外围功能还是会有些差异，比如窗口没有了圆角效果，而且会多了系统标题栏，这种问题我觉得可以接受，因为圆角效果、窗口标题栏可以认为是DDE所提供的功能，而不是应用程序提供的。

我觉得现在的操作步骤应该是，对于每个应用，确认在非 DDE 环境下基本功能可以正常运行再执行此次对 XDG_CURRENT_DESKTOP 判断逻辑的删除。否则会让应用重新面临在 DDE 以外基本功能无法正常使用的问题（如 #5723 中描述的，当前版本的 deepin-terminal）。