make xlib optional for wayland (slgobinath#765)

* refactor(break_screen): make Xlib not required for Wayland

* refactor(donotdisturb): make Xlib not required for Wayland

* move dependency on Xlib to optional/recommended

Author: deltragon

README.md

@@ -122,10 +122,10 @@ Ensure to meet the following dependencies:
 - `ffmpeg` or `pipewire` (any of them works)
 - `python3` (>= 3.10.0)
 - `python3-gi`
-- `python3-xlib`
 - `python3-babel`
 - `python3-croniter`
 - `python3-packaging`
+- `python3-xlib` (required on x11)
 - **Optional**: Either `python3-pywayland` (provides smartpause in Wayland) or `xprintidle` (provides smartpause in x11).
 
 **To install Safe Eyes from PyPI:**

debian/control

@@ -12,14 +12,14 @@ Architecture: all
 Depends: ${misc:Depends}, ${python3:Depends}, 
  python3 (>= 3.10.0),
  python3-gi,
- python3-xlib,
  python3-babel,
  python3-croniter,
  python3-packaging,
  gir1.2-notify-0.7,
  gir1.2-gtk-4.0,
  ffmpeg | pipewire
 Recommends:
+ python3-xlib,
  python3-pywayland
 Suggests:
  xprintidle

safeeyes/plugins/donotdisturb/plugin.py

@@ -26,7 +26,6 @@
 
 gi.require_version("Gio", "2.0")
 from gi.repository import Gio
-import Xlib
 from safeeyes import utility
 
 context = None
@@ -63,6 +62,8 @@ def is_active_window_skipped_xorg(pre_break):
     """
     logging.info("Searching for full-screen application")
 
+    import Xlib
+
     def get_window_property(window, prop, proptype):
         result = window.get_full_property(prop, proptype)
         if result:

safeeyes/ui/break_screen.py

@@ -27,9 +27,6 @@
 from safeeyes.context import Context
 from safeeyes.model import Break, Config, TrayAction
 from safeeyes.translations import translate as _
-import Xlib
-from Xlib.display import Display
-from Xlib import X
 
 gi.require_version("Gtk", "4.0")
 from gi.repository import Gdk
@@ -71,7 +68,9 @@ def __init__(
         self.show_postpone_button = False
 
         if not self.context.is_wayland:
-            self.x11_display = Display()
+            import Xlib.display
+
+            self.x11_display = Xlib.display.Display()
 
     def initialize(self, config: Config) -> None:
         """Initialize the internal properties from configuration."""
@@ -231,6 +230,8 @@ def __window_set_keep_above_x11(self, window: "BreakScreenWindow") -> None:
         if self.x11_display is None:
             return
 
+        import Xlib
+
         NET_WM_STATE = self.x11_display.intern_atom("_NET_WM_STATE")
         NET_WM_STATE_ABOVE = self.x11_display.intern_atom("_NET_WM_STATE_ABOVE")
         NET_WM_STATE_STICKY = self.x11_display.intern_atom("_NET_WM_STATE_STICKY")
@@ -277,6 +278,8 @@ def __lock_keyboard_x11(self) -> None:
         if self.x11_display is None:
             return
 
+        from Xlib import X
+
         logging.info("Lock the keyboard")
         self.lock_keyboard = True