Opened 5 years ago

Closed 5 years ago

#12438 closed Bug Report - General (fixed)

xscreensaver (XFCE) inhibit is broken after the latest dbus screen saver commits

Reported by: tomi.orava@… Owned by: Stuart Auchterlonie
Priority: major Milestone: 0.28
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

The latest dbus screen saver commits broke the Ubuntu 14.04 + xfce + xscreensaver combination and now the screen saver inhibit functionality doesn't work at all.

Also the lirc remote control broke down, but there seems to be another ticket for that.

commit 35564d70e71ee0fe3ccea7dfa53a1ffa21117057 Author: Stuart Auchterlonie <stuarta@…> Date: Thu Mar 19 21:07:47 2015 +0000

Initial version of dbus based screensaver

My previous & working client version was 2864a79b78335e899e00e49d5b76e4f7d13f0f13, just before the dbus screen saver support was added.

I'm currently away from the host in question, but I can provide logs later on in the evening (ie. after 8h) if necessary.

Change History (6)

comment:1 Changed 5 years ago by Stuart Auchterlonie

Milestone: unknown0.28
Owner: set to Stuart Auchterlonie
Priority: minormajor
Status: newassigned

comment:2 Changed 5 years ago by Stuart Auchterlonie

This (the xscreensaver part) should be relatively easy to fix. The existing code used both dpms controls as well as xscreensaver. I didn't plumb that back in, so I see what I can do here.

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

In b1c3c35a99fbd3498a01dc06888801b553604927/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.

Refs #12438
Refs #12414

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

In 348ed663810ffa6860f380733a677e0aa6d729d1/mythtv:

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

Refs #12438
Refs #12414

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

In e731ebc0d31d20afa29a95a3934be0149c2e0383/mythtv:

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

Refs #12438
Refs #12414

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

Resolution: fixed
Status: assignedclosed

In 8ea5911f066fe406deeddb8ba92b629be4f47cd1/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.

Fixes #12438
Refs #12414

Note: See TracTickets for help on using tickets.