doc: add information on rules under Wayland and uinput permissions

docs/index.md

@@ -152,6 +152,11 @@ for the step-by-step procedure for manual installation.
   To make the Polling Rate and M-Key LEDs settings effective the Onboard Profiles setting has to be disabled.
   This may have other effects, such as turning off backlighting.
 
+- Solaar will try to use uinput to simulate input from rules if Xtest is not available
+  but this needs write permission on /dev/uinput.
+  For more information see [the rules page](https://pwr-solaar.github.io/Solaar/rules).
+
+
 ## Contributing to Solaar
 
 Conributions to Solaaar are very welcome.

docs/installation.md

@@ -22,7 +22,7 @@ The `udev` package must be installed and its daemon running.
 
 Solaar requires Python 3.6+ and requires several packages to be installed.
 If you are running the system version of Python you should have the
-`python3-pyudev`, `python3-psutil`, `python3-xlib`,
+`python3-pyudev`, `python3-psutil`, `python3-xlib`, `python3-evdev`,
 and `python3-yaml` or `python3-pyyaml` packages installed.
 To run the GUI Solaar also requires Gtk3 and its GObject introspection bindings.
 If you are running the system version of Python

docs/rules.md

@@ -5,12 +5,17 @@ layout: page
 
 Creating and editing most rules can be done in the Solaar GUI, by pressing the 'Rule Editor' button in the
 Solaar main window.
+
 Rule processing is an experimental feature.  Significant changes might be made in response to problems.
 
-Note that rule processing only fully works under X11.
+*Note that rule processing only fully works under X11.
-When running under Wayland with X11 libraries loaded most features will not be available and errors may result.
+When running under Wayland with X11 libraries loaded some features will not be available.
-Features known not to work under Wayland include process conditions and
+When running under Wayland without X11 libraries loaded even more features will not be available.
-anything to do with simulating keyboard or mouse input.
+Rule features known not to work under Wayland include process and mouse process conditions.
+Under Wayland using keyboard groups may result in incorrect symbols being input for simulated input.
+Under Wayland simulating inputs when modifier keys are pressed may result in incorrect symbols being input.
+Simulated input uses Xtest if available or uinput if the user has write access to /dev/uinput.
+To get access to /dev/uinput run `sudo setfacl -m u:${user}:rw /dev/uinput`*
 
 Logitech devices that use HID++ version 2.0 or greater produce feature-based
 notifications that Solaar can process using a simple rule language.  For
@@ -20,17 +25,17 @@ which normally does not produce any input at all when the keyboard is in
 Windows mode.
 
 Solaar's rules only trigger on HID++ notifications so device actions that
-normally produce HID output need rule processing have to be first be set to
+normally produce HID output have to be first be set (diverted) to
-produce HID++ notifications instead of their normal behavior (diverted).
+produce HID++ notifications instead of their normal behavior.
 Currently Solaar can divert some mouse scroll wheels, some
 mouse thumb wheels, the crown of Craft keyboards, and some keys and buttons.
 If the scroll wheel, thumb wheel, crown, key, or button is
 not diverted by setting the appropriate setting then no HID++ notification is
-generated so rules will not be triggered by manipulating the wheel, crown, key, or button.
+generated and rules will not be triggered by manipulating the wheel, crown, key, or button.
 Look for `HID++` or `Diversion` settings to see what
 diversion can be done with your devices.
 
-Running Solaar with the `-dd`
+Running Solaar with the `-ddd`
 option will show information about notifications, including their feature
 name, report number, and data.
 
@@ -139,6 +144,17 @@ Another example would be mapping `Back Button` -> `Back Button`. With this one,
 Mouse movements and buttons can be mixed and chained together however you like.
 It's possible to create a `No-op` gesture by clicking 'Delete' on the initial Action when you first create the rule. This gesture will trigger when you simply click the 'Gesture' button.
 
+`Setting` conditions check device settings of devices, provided the device is on-line.
+The first arguments to the condition are the Serial number or Unit ID of a device, as shown in Solaar's detail pane,
+or null for the device that initiated rule processing; and
+the internal name of a setting (which can be found from solaar config <device>).
+Most simple settings take one extra argument, the value to check the setting value against.
+Range setting can also take two arguments, which form an inclusive range to check against.
+Other settings take two arguments, a key indicating which sub-setting to check and the value to check it against.
+For settings that use gestures as an argument the internal name of the gesture is used,
+which can be found in the GESTURE2_GESTURES_LABELS structure in lib/logitech_receiver/settings_templates.
+For boolean settings '~' can be used to toggle the setting.
+
 A `KeyPress` action takes a sequence of X11 key symbols and simulates a chorded keypress on the keyboard, such as "A", "Shift+A", or "Control+A".
 Any key symbols that correspond to modifier keys that are in the current keyboard modifiers are ignored.
 Use separate  `KeyPress` actions for multiple characters.
@@ -148,19 +164,15 @@ A `MouseClick` action takes a mouse button name (`left`, `middle` or `right`) an
 An `Execute` action takes a program and arguments and executes it asynchronously.
 
 A `Set` action changes a Solaar setting for a device, provided that the device is on-line.
-`Set` actions take three or four arguments, depending on the setting:
+`Set` actions take three or four arguments, depending on the setting.
-the Serial number or Unit ID of a device, as shown in Solaar's detail pane,
+The first two are the Serial number or Unit ID of a device, as shown in Solaar's detail pane,
-or null for the device that initiated rule processing;
+or null for the device that initiated rule processing; and
-the internal name of a setting (which can be found from solaar config <device>);
+the internal name of a setting (which can be found from solaar config <device>).
-one or two arguments for the setting.
+Simple settings take one extra argument, the value to set the setting to.
-For settings that use keys or buttons as an argument the Logtech name can be used
+For boolean settings '~' can be used to toggle the setting.
-as shown in the Solaar main window for these settings,
+Other simple settings take two extra arguments, a key indicating which sub-setting to set and the value to set it to.
-or the numeric value for the key or button.
 For settings that use gestures as an argument the internal name of the gesture is used,
 which can be found in the GESTURE2_GESTURES_LABELS structure in lib/logitech_receiver/settings_templates.
-For settings that need one of a set of names as an argument the name can be used or its internal integer value,
-as used in the Solaar config file.
-For boolean settings '~' can be used to toggle the setting.
 All settings are supported.
 
 Solaar has several built-in rules, which are run after user-created rules and so can be overridden by user-created rules.

lib/logitech_receiver/diversion.py

@@ -101,7 +101,7 @@ try:
     Xkbdisplay = X11Lib.XOpenDisplay(None)
 except Exception:
     _log.warn(
-        'X11 not available - rules cannot access current process or keyboard group and cannot simulate input. %s',
+        'X11 not available - rules cannot access current process, modifier keys, or keyboard group. %s',
         exc_info=_sys.exc_info()
     )
     modifier_keycodes = []
@@ -914,7 +914,7 @@ COMPONENTS = {
 }
 
 built_in_rules = Rule([])
-if True:  # x11
+if True:
     built_in_rules = Rule([
         {'Rule': [  # Implement problematic keys for Craft and MX Master
             {'Rule': [{'Key': ['Brightness Down', 'pressed']}, {'KeyPress': 'XF86_MonBrightnessDown'}]},
@@ -952,8 +952,6 @@ mr_key_down = False
 
 # process a notification
 def process_notification(device, status, notification, feature):
-    if False:  # not x11
-        return
     global keys_down, g_keys_down, m_keys_down, mr_key_down, key_down, key_up
     key_down, key_up = None, None
     # need to keep track of keys that are down to find a new key down
@@ -1070,5 +1068,4 @@ def _load_config_rule_file():
     rules = Rule([Rule(loaded_rules, source=_file_path), built_in_rules])
 
 
-if True:  # x11
+_load_config_rule_file()
-    _load_config_rule_file()