Opened 4 years ago

Closed 4 years ago

#12511 closed Bug Report - General (fixed)

dbus based screensaver for 0.27

Reported by: Stuart Auchterlonie Owned by: Stuart Auchterlonie
Priority: minor Milestone: 0.27.6
Component: MythTV - General Version: 0.27.5
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Ticket to track backporting #12414 to 0.27

Change History (9)

comment:1 Changed 4 years ago by Stuart Auchterlonie <stuarta@…>

In ac595a7e69cb8cb13d9d43cab0cb2fef36ec8dbf/mythtv:

Initial version of dbus based screensaver

(cherry picked from commit 35564d70e71ee0fe3ccea7dfa53a1ffa21117057)

Backport to fixes/0.27. Refs #12511

comment:2 Changed 4 years ago by Stuart Auchterlonie <stuarta@…>

In 7c7c4e5dd0ea7795414e44da5170662d283c81a5/mythtv:

Attempt to use 4 different known screensaver dbus services

(cherry picked from commit eb68676d8ea4dae3cc2d823d2ba10183b00a4680)

Backport to fixes/0.27. Refs #12511

comment:3 Changed 4 years ago by Stuart Auchterlonie <stuarta@…>

In 31bd913580153c7b7ab4ce32850d8bc0c4298634/mythtv:

lost a void along the way

(cherry picked from commit b316010a65a70561852f34ae2891825c8189d76b)

Backport to fixes/0.27. Refs #12511

comment:4 Changed 4 years ago by Stuart Auchterlonie <stuarta@…>

In 42939f201a653b8e66876ba88b2bcbc4b57511a2/mythtv:

Refs #12414. Removes some old commented out code

(cherry picked from commit 7c18c182e693302915bfcdd483f95192c95c04f1)

Backport to fixes/0.27. Refs #12511

comment:5 Changed 4 years ago by Stuart Auchterlonie <stuarta@…>

In 1b02deb1b9fd89ed27d2f98bb00137ff48d60bf7/mythtv:

Fix lirc being ignored after dbus screen saver changes.

If the Screensaver is Asleep() we ignore all keypresses
generated via the remote. This is clearly so that a button
pressed on the remote while the screen is asleep only wakes
up the screen, rather than triggering an action.

This swaps the logic around in the dbus screensaver so that
it satisfies the Asleep() check.

We may need to revist this in the future if we find that
when using the dbus screensaver we are not ignoring the
remote keypress which is used to bring us out of screensave.

(cherry picked from commit 70ae25deab026238250cbe0ca108fb6ceb41e188)

Backport to fixes/0.27. Refs #12511

comment:6 Changed 4 years ago by Stuart Auchterlonie <stuarta@…>

In 43312ef43893b0a720b9c49c6b1d1112c34c79e9/mythtv:

Separate the ScreenSaver? base class from the ScreenSaverController?.

This is a preliminary step to allow multiple screen saver control
methods to be utilized at the same time.

(cherry picked from commit b1c3c35a99fbd3498a01dc06888801b553604927)

Backport to fixes/0.27. Refs #12511

comment:7 Changed 4 years ago by Stuart Auchterlonie <stuarta@…>

In 05016ef186d113a0058babe4002a429ffc2f035e/mythtv:

Move failure to connect to the different dbus services to LOG_WARNING level

(cherry picked from commit 348ed663810ffa6860f380733a677e0aa6d729d1)

Backport to fixes/0.27. Refs #12511

comment:8 Changed 4 years ago by Stuart Auchterlonie <stuarta@…>

In 156c30034f6d0c2c22c3b61f59da075f50938269/mythtv:

Move failure to connect to the different dbus services to LOG_WARNING level #2

(cherry picked from commit e731ebc0d31d20afa29a95a3934be0149c2e0383)

Backport to fixes/0.27. Refs #12511

comment:9 Changed 4 years ago by Stuart Auchterlonie <stuarta@…>

Resolution: fixed
Status: newclosed

In 2ad315894af9a7770b29dbec5d0ca56a84534f49/mythtv:

Re-Enable screensaver-x11 after the dbus screensaver support was added.

This now utilizes the dbus screensaver first, and then the previously
existing screensaver methods. This should resolve the issues seen by
those using only DPMS or xscreensaver.

(cherry picked from commit 8ea5911f066fe406deeddb8ba92b629be4f47cd1)

Backport to fixes/0.27. Fixes #12511

Note: See TracTickets for help on using tickets.