Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable dbus activation of xdg-desktop-portal until the dbus environment is updated #496

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

q234rty
Copy link

@q234rty q234rty commented Jan 13, 2024

QT seems to automatically call xdg-desktop-portal via dbus 1, which might cause xdg-desktop-portal to be dbus activated before we sync our environment to dbus, breaking portal selection via XDG_CURRENT_DESKTOP.

Opt out 2 of their portal usage until updateDBusEnvironment finishes.

See also https://invent.kde.org/plasma/plasma-workspace/-/commit/670cf731a75037c646864ed0bd03a35a35130c5e

Fixes lxqt/xdg-desktop-portal-lxqt#23

…ent is updated

QT seems to automatically call xdg-desktop-portal via dbus [1],
which might cause xdg-desktop-portal to be dbus activated before
we sync our environment to dbus, breaking portal selection via
XDG_CURRENT_DESKTOP.

Opt out [2] of their portal usage until `updateDBusEnvironment` finishes.

See also https://invent.kde.org/plasma/plasma-workspace/-/commit/670cf731a75037c646864ed0bd03a35a35130c5e

[1]: https://invent.kde.org/qt/qt/qtbase/-/commit/9d65da811207992a97c5391d1e2e1981f8bae114
[2]: https://invent.kde.org/qt/qt/qtbase/-/commit/b4b7f6f5a5636c7ded0c8285fce8d41d0ab30b29
@stefonarch
Copy link
Member

stefonarch commented Jan 13, 2024

Can confirm that on archxdg-desktop-portal-lxqt is not working when booting into x11.
Booting in labwc is fine instead, debian booting into x11 too.

@tsujan tsujan requested a review from palinek January 14, 2024 13:30
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.

firefox needs .conf file manually created to work
2 participants