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

Inhibit screensaver in a later initialization stage #15094

Merged
merged 1 commit into from
Dec 20, 2018

Conversation

yol
Copy link
Member

@yol yol commented Dec 19, 2018

When the OS screensaver is already inhibited in InitializeGUI, e.g. the
language is not loaded yet, which has led to problems on GNOME because
the screensaver inhibition requires a translated "reason" string.

Moving the inhibition to a later stage solves this.

When the OS screensaver is already inhibited in InitializeGUI, e.g. the
language is not loaded yet, which has led to problems on GNOME because
the screensaver inhibition requires a translated "reason" string.

Moving the inhibition to a later stage solves this.
@yol yol self-assigned this Dec 19, 2018
@yol yol added Type: Fix non-breaking change which fixes an issue Platform: Linux v18 Leia labels Dec 19, 2018
@yol
Copy link
Member Author

yol commented Dec 19, 2018

@MartijnKaijser low-risk and solves > 20 s startup delay on Linux when Kodi has screensaver set

@MartijnKaijser MartijnKaijser added this to the Leia 18.0-rc4 milestone Dec 19, 2018
@yol
Copy link
Member Author

yol commented Dec 19, 2018

jenkins build and merge

1 similar comment
@yol
Copy link
Member Author

yol commented Dec 20, 2018

jenkins build and merge

@yol
Copy link
Member Author

yol commented Dec 20, 2018

Build failures unrelated

@yol yol merged commit 817e7a8 into master Dec 20, 2018
@yol yol deleted the late-inhibit-screensaver branch December 20, 2018 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Platform: Linux Type: Fix non-breaking change which fixes an issue v18 Leia
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants