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

session-management: mention dbus-update-activation-environment #626

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

Conversation

ericonr
Copy link
Member

@ericonr ericonr commented Nov 29, 2021

Necessary, for example, to allow gnome-keyring-daemon to launch prompts
for user interaction, since the prompts are launched by the dbus daemon.

Before submitting a pull request, please read CONTRIBUTING; pull requests that do not meet the criteria described there will not be merged. Note that this repository's CONTRIBUTING contains information specific to this repository, and is not the same as CONTRIBUTING for the void-packages repository.

We prioritise pull requests involving information specific to Void over those involving information applicable to Linux in general.

Necessary, for example, to allow gnome-keyring-daemon to launch prompts
for user interaction, since the prompts are launched by the dbus daemon.
@ericonr
Copy link
Member Author

ericonr commented Nov 29, 2021

@steinex iirc we talked about this when I mentioned my service that had a sleep 4 in the middle :p

Does it look ok?

@steinex
Copy link

steinex commented Nov 29, 2021

I think it's not actionable enough - imo audience should be told to run it as part of their (sway|X11 wm) config so that it's called after their Wayland/X session is up.

Then for sway, you also want XDG_CURRENT_DESKTOP=sway (but I forgot why and i'm not sure if is important to mention / in scope, iirc it was for portal-wlr).

Other than that: thanks! :-)

@ericonr
Copy link
Member Author

ericonr commented Nov 29, 2021

Thanks, I will try to include some guidance on when to run it.

Portal-wlr does need the current desktop var, I will try to sneak in something about "other applicable variables" (though for my usage I just launch dbus-run-session with that one set already).

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.

2 participants