Skip to content

Start after Gala#79

Draft
meisenzahl wants to merge 2 commits into
mainfrom
start-after-gala
Draft

Start after Gala#79
meisenzahl wants to merge 2 commits into
mainfrom
start-after-gala

Conversation

@meisenzahl

Copy link
Copy Markdown
Member

Closes #78

@meisenzahl

Copy link
Copy Markdown
Member Author

@Marukesu

Copy link
Copy Markdown
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
@meisenzahl
meisenzahl requested a review from danirabbit May 24, 2023 19:26
@danirabbit

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

Tried here without success, the delay continues.

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

tintou commented May 25, 2023

Copy link
Copy Markdown
Member

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