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

Start after Gala #79

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Start after Gala #79

wants to merge 2 commits into from

Conversation

meisenzahl
Copy link
Member

Closes #78

@meisenzahl
Copy link
Member Author

@Marukesu
Copy link
Contributor

i looked at what GNOME does in they backend and it's seem to use graphical-session.target.

@meisenzahl meisenzahl marked this pull request as ready for review May 24, 2023 19:26
@danirabbit
Copy link
Member

Okay so with the latest commit the session doesn’t crash like it did before but it takes a REALLY long time to start. So I think there’s probably still some issue here.

@danirabbit
Copy link
Member

I wonder if we're waiting on Portals in Gala because we use access dialog: https://github.com/elementary/gala/blob/d408d394dbd9a6c6999df924b9c47599cea58c15/src/Dialogs.vala#L7

@Marukesu
Copy link
Contributor

now the issue is the portal frontend trying to start the backend and falling systemctl --user status xdg-desktop-portal shows:

Failed to create app chooser proxy: Erro ao chamar StartServiceByName para org.freedesktop.impl.portal.desktop.pantheon: O tempo limite foi alcançado

@davidmhewitt
Copy link
Member

There are two synchronous DBus connections (to gala, and the notifications daemon) being made in the constructors of the new Background portal code.

These should probably be watches instead of blocking the startup of the portal process, and it would allow the connections to be re-made if either gala or the notification daemon crashed or restarted.

Not sure if that's the cause of the long session load here, but we had a similar issue in the past, solved with this:
https://github.com/elementary/portals/pull/34/files

@Marukesu
Copy link
Contributor

Tried here without success, the delay continues.

@meisenzahl meisenzahl marked this pull request as draft May 24, 2023 20:25
@tintou
Copy link
Member

tintou commented May 25, 2023

I believe that we should first switch to using systemd to start the session (It's bug elementary/session-settings#17)

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.

Portal sometimes undecorated
5 participants