[Portland-bugs] [Bug 97811] New: Warn / fail if xdg-screensaver cannot find xprop

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=97811

            Bug ID: 97811
           Summary: Warn / fail if xdg-screensaver cannot find xprop
           Product: Portland
           Version: 1.1.0
          Hardware: x86-64 (AMD64)
                OS: Linux (All)
            Status: NEW
          Severity: normal
          Priority: medium
         Component: xdg-utils
          Assignee: portland-bugs at lists.freedesktop.org
          Reporter: nicolas+freedesktop at guillaumin.me

On Gnome 3, xdg-screensaver fails to suspend the screensaver if xprop is not
installed.

Without xprop the $screensaver_file is not created (as xdg-screensaver cannot
retrieve the window ID), and so the "inhibitor" placed on the session via DBus
expires after the 10s sleep in the Perl DBus script loop.

---
while (1) {                                                                     
  sleep(10);                                                                    
  my $status = new IO::File($screensaver_file, "r")                             
    or exit 0;                       
---

xdg-screensaver seems to try to support the case where xprop is not installed
(It checks for xprop, and various functions bail out early if it's not
available). However with Gnome 3 (and perhaps with other environments) it seems
it's a hard requirement, and as such the user should be warned if it's missing.

I believe that affects numerous users as there have been several reports of
xdg-screensaver failing with Gnome in various projects:

* https://bugzilla.redhat.com/show_bug.cgi?id=665918
* https://trac.videolan.org/vlc/ticket/4739
* https://forum.videolan.org/viewtopic.php?t=127389

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/portland-bugs/attachments/20160915/6a1f6e09/attachment.html>