Skip to content

Navigation Menu

Sign in
Sign up

Start after Gala - #79

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

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

Conversation

@meisenzahl

@meisenzahl meisenzahl commented May 24, 2023

Copy link
Copy Markdown
Member

Closes #78

Copy link
Copy Markdown
Member Author

Copy link
Copy Markdown
Contributor

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

danirabbit and meisenzahl reacted with heart emoji

meisenzahl marked this pull request as ready for review May 24, 2023 19:26

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.

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

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

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

meisenzahl reacted with thumbs up emoji

Copy link
Copy Markdown
Contributor

Tried here without success, the delay continues.

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

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)

danirabbit reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danirabbit danirabbit Awaiting requested review from danirabbit

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Portal sometimes undecorated

AltStyle によって変換されたページ (->オリジナル) /