From 21dc595f32c88f02cb47015546d4c3af0d725d38 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tomasz=20Paku=C5=82a?= Date: Tue, 12 Nov 2024 16:33:29 +0100 Subject: [PATCH 01/40] Move `syn()` convenience method from `InputDevice` to `EventIO` (#224) Move `syn()` method from `UInput` to `EventIO`, makes it possible to use it with `InputDevice` as well --- evdev/eventio.py | 9 +++++++++ evdev/uinput.py | 9 --------- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/evdev/eventio.py b/evdev/eventio.py index 1b0e5cc..415e2e8 100644 --- a/evdev/eventio.py +++ b/evdev/eventio.py @@ -136,5 +136,14 @@ def write(self, etype, code, value): _uinput.write(self.fd, etype, code, value) + def syn(self): + """ + Inject a ``SYN_REPORT`` event into the input subsystem. Events + queued by :func:`write()` will be fired. If possible, events + will be merged into an 'atomic' event. + """ + + self.write(ecodes.EV_SYN, ecodes.SYN_REPORT, 0) + def close(self): pass diff --git a/evdev/uinput.py b/evdev/uinput.py index 476a84a..c4225d8 100644 --- a/evdev/uinput.py +++ b/evdev/uinput.py @@ -227,15 +227,6 @@ def close(self): _uinput.close(self.fd) self.fd = -1 - def syn(self): - """ - Inject a ``SYN_REPORT`` event into the input subsystem. Events - queued by :func:`write()` will be fired. If possible, events - will be merged into an 'atomic' event. - """ - - _uinput.write(self.fd, ecodes.EV_SYN, ecodes.SYN_REPORT, 0) - def capabilities(self, verbose=False, absinfo=True): """See :func:`capabilities `.""" if self.device is None: From d182b7fbd145a245214a3a1949c0f85d38b18cf5 Mon Sep 17 00:00:00 2001 From: dani-hs Date: Sun, 19 Jan 2025 05:17:33 +0100 Subject: [PATCH 02/40] Fix swapped delay and repeat (#227) --- evdev/device.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/evdev/device.py b/evdev/device.py index cde168e..758f899 100644 --- a/evdev/device.py +++ b/evdev/device.py @@ -17,7 +17,7 @@ # -------------------------------------------------------------------------- _AbsInfo = collections.namedtuple("AbsInfo", ["value", "min", "max", "fuzz", "flat", "resolution"]) -_KbdInfo = collections.namedtuple("KbdInfo", ["repeat", "delay"]) +_KbdInfo = collections.namedtuple("KbdInfo", ["delay", "repeat"]) _DeviceInfo = collections.namedtuple("DeviceInfo", ["bustype", "vendor", "product", "version"]) @@ -70,16 +70,16 @@ class KbdInfo(_KbdInfo): Attributes ---------- - repeat - Keyboard repeat rate in characters per second. - delay Amount of time that a key must be depressed before it will start to repeat (in milliseconds). + + repeat + Keyboard repeat rate in characters per second. """ def __str__(self): - return "repeat {}, delay {}".format(*self) + return "delay {}, repeat {}".format(*self) class DeviceInfo(_DeviceInfo): From b1a5bd1cdf2dd8294c18ae97f7c1902b639c99b3 Mon Sep 17 00:00:00 2001 From: Tobi <28510156+sezanzeb@users.noreply.github.com> Date: Sun, 19 Jan 2025 10:37:30 +0100 Subject: [PATCH 03/40] Add pylint -E and pytest to ci (#228) * Remove EOL python 3.7 from the ci * Add pylint -E, fix some pylint errors * Add pytest step * Fix test_abs_values * Turned RuntimeError into the desired UInputError if the device is not a character device, caused by a re-raise outside an except block * Add test for S_ISCHR False --- .github/workflows/install.yaml | 4 ++-- .github/workflows/lint.yml | 27 +++++++++++++++++++++++++++ .github/workflows/test.yml | 29 +++++++++++++++++++++++++++++ evdev/ecodes.py | 1 + evdev/eventio.py | 2 ++ evdev/events.py | 24 ++++++++++++------------ evdev/uinput.py | 2 +- tests/test_uinput.py | 20 ++++++++++++++------ 8 files changed, 88 insertions(+), 21 deletions(-) create mode 100644 .github/workflows/lint.yml create mode 100644 .github/workflows/test.yml diff --git a/.github/workflows/install.yaml b/.github/workflows/install.yaml index 7d965b2..f959de2 100644 --- a/.github/workflows/install.yaml +++ b/.github/workflows/install.yaml @@ -11,10 +11,10 @@ jobs: fail-fast: false matrix: os: [ubuntu-latest] - python-version: ["3.7", "3.8", "3.9", "3.10", "3.11", "3.12"] + python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"] include: - os: ubuntu-latest - python-version: "3.7" + python-version: "3.8" steps: - uses: actions/checkout@v4 diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml new file mode 100644 index 0000000..d499462 --- /dev/null +++ b/.github/workflows/lint.yml @@ -0,0 +1,27 @@ +name: Lint + +on: + - push + - pull_request + +jobs: + pylint: + runs-on: ${{ matrix.os }} + strategy: + fail-fast: false + matrix: + os: [ubuntu-latest] + python-version: ["3.12"] + + steps: + - uses: actions/checkout@v4 + - name: Set up Python ${{ matrix.python-version }} + uses: actions/setup-python@v5 + with: + python-version: ${{ matrix.python-version }} + + - name: Check for pylint errors + run: | + python -m pip install pylint setuptools + python setup.py build + python -m pylint --disable=no-member --verbose -E build/lib*/evdev diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml new file mode 100644 index 0000000..3ee56d3 --- /dev/null +++ b/.github/workflows/test.yml @@ -0,0 +1,29 @@ +name: Test + +on: + - push + - pull_request + +jobs: + test: + runs-on: ${{ matrix.os }} + strategy: + fail-fast: false + matrix: + os: [ubuntu-latest] + python-version: ["3.12"] + + steps: + - uses: actions/checkout@v4 + - name: Set up Python ${{ matrix.python-version }} + uses: actions/setup-python@v5 + with: + python-version: ${{ matrix.python-version }} + + - name: Run pytest tests + # pip install -e . builds _ecodes and such into the evdev directory + # sudo required to write to uinputs + run: | + sudo python -m pip install pytest setuptools + sudo python -m pip install -e . + sudo python -m pytest tests diff --git a/evdev/ecodes.py b/evdev/ecodes.py index 3562368..759cfe7 100644 --- a/evdev/ecodes.py +++ b/evdev/ecodes.py @@ -1,3 +1,4 @@ +# pylint: disable=undefined-variable """ This modules exposes the integer constants defined in ``linux/input.h`` and ``linux/input-event-codes.h``. diff --git a/evdev/eventio.py b/evdev/eventio.py index 415e2e8..3335500 100644 --- a/evdev/eventio.py +++ b/evdev/eventio.py @@ -72,6 +72,7 @@ def read(self): for event in events: yield InputEvent(*event) + # pylint: disable=no-self-argument def need_write(func): """ Decorator that raises :class:`EvdevError` if there is no write access to the @@ -82,6 +83,7 @@ def need_write(func): def wrapper(*args): fd = args[0].fd if fcntl.fcntl(fd, fcntl.F_GETFL) & os.O_RDWR: + # pylint: disable=not-callable return func(*args) msg = 'no write access to device "%s"' % args[0].path raise EvdevError(msg) diff --git a/evdev/events.py b/evdev/events.py index 104b563..97f570d 100644 --- a/evdev/events.py +++ b/evdev/events.py @@ -65,13 +65,13 @@ def timestamp(self): """Return event timestamp as a float.""" return self.sec + (self.usec / 1000000.0) - def __str__(s): + def __str__(self): msg = "event at {:f}, code {:02d}, type {:02d}, val {:02d}" - return msg.format(s.timestamp(), s.code, s.type, s.value) + return msg.format(self.timestamp(), self.code, self.type, self.value) - def __repr__(s): + def __repr__(self): msg = "{}({!r}, {!r}, {!r}, {!r}, {!r})" - return msg.format(s.__class__.__name__, s.sec, s.usec, s.type, s.code, s.value) + return msg.format(self.__class__.__name__, self.sec, self.usec, self.type, self.code, self.value) class KeyEvent: @@ -119,8 +119,8 @@ def __str__(self): msg = "key event at {:f}, {} ({}), {}" return msg.format(self.event.timestamp(), self.scancode, self.keycode, ks) - def __repr__(s): - return "{}({!r})".format(s.__class__.__name__, s.event) + def __repr__(self): + return "{}({!r})".format(self.__class__.__name__, self.event) class RelEvent: @@ -136,8 +136,8 @@ def __str__(self): msg = "relative axis event at {:f}, {}" return msg.format(self.event.timestamp(), REL[self.event.code]) - def __repr__(s): - return "{}({!r})".format(s.__class__.__name__, s.event) + def __repr__(self): + return "{}({!r})".format(self.__class__.__name__, self.event) class AbsEvent: @@ -153,8 +153,8 @@ def __str__(self): msg = "absolute axis event at {:f}, {}" return msg.format(self.event.timestamp(), ABS[self.event.code]) - def __repr__(s): - return "{}({!r})".format(s.__class__.__name__, s.event) + def __repr__(self): + return "{}({!r})".format(self.__class__.__name__, self.event) class SynEvent: @@ -173,8 +173,8 @@ def __str__(self): msg = "synchronization event at {:f}, {}" return msg.format(self.event.timestamp(), SYN[self.event.code]) - def __repr__(s): - return "{}({!r})".format(s.__class__.__name__, s.event) + def __repr__(self): + return "{}({!r})".format(self.__class__.__name__, self.event) #: A mapping of event types to :class:`InputEvent` sub-classes. Used diff --git a/evdev/uinput.py b/evdev/uinput.py index c4225d8..756f83c 100644 --- a/evdev/uinput.py +++ b/evdev/uinput.py @@ -272,7 +272,7 @@ def _verify(self): try: m = os.stat(self.devnode)[stat.ST_MODE] if not stat.S_ISCHR(m): - raise + raise OSError except (IndexError, OSError): msg = '"{}" does not exist or is not a character device file ' "- verify that the uinput module is loaded" raise UInputError(msg.format(self.devnode)) diff --git a/tests/test_uinput.py b/tests/test_uinput.py index 2bf3dc1..dcd09e0 100644 --- a/tests/test_uinput.py +++ b/tests/test_uinput.py @@ -1,10 +1,12 @@ # encoding: utf-8 - +import stat from select import select -from pytest import raises, fixture +from unittest.mock import patch -from evdev import uinput, ecodes, events, device, util +import pytest +from pytest import raises, fixture +from evdev import uinput, ecodes, device, UInputError # ----------------------------------------------------------------------------- uinput_options = { @@ -66,12 +68,12 @@ def test_enable_events(c): def test_abs_values(c): e = ecodes - c["events"] = { + c = { e.EV_KEY: [e.KEY_A, e.KEY_B], - e.EV_ABS: [(e.ABS_X, (0, 255, 0, 0)), (e.ABS_Y, device.AbsInfo(0, 255, 5, 10, 0, 0))], + e.EV_ABS: [(e.ABS_X, (0, 0, 255, 0, 0)), (e.ABS_Y, device.AbsInfo(0, 0, 255, 5, 10, 0))], } - with uinput.UInput(**c) as ui: + with uinput.UInput(events=c) as ui: c = ui.capabilities() abs = device.AbsInfo(value=0, min=0, max=255, fuzz=0, flat=0, resolution=0) assert c[e.EV_ABS][0] == (0, abs) @@ -114,3 +116,9 @@ def test_write(c): assert evs[3].code == ecodes.KEY_A and evs[3].value == 2 assert evs[4].code == ecodes.KEY_A and evs[4].value == 0 break + + +@patch.object(stat, 'S_ISCHR', return_value=False) +def test_not_a_character_device(c): + with pytest.raises(UInputError): + uinput.UInput(**c) From 047bf13da1dc5acc3573f2d66c7d378011e73ec8 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Mon, 20 Jan 2025 21:32:02 +0100 Subject: [PATCH 04/40] Use relative imports and sort imports --- evdev/__init__.py | 11 +++++------ evdev/device.py | 11 +++++------ evdev/ecodes.py | 2 +- evdev/eventio.py | 8 ++++---- evdev/eventio_async.py | 4 ++-- evdev/events.py | 2 +- evdev/evtest.py | 13 ++++--------- evdev/ff.py | 2 +- evdev/uinput.py | 8 +++----- evdev/util.py | 10 +++++----- 10 files changed, 31 insertions(+), 40 deletions(-) diff --git a/evdev/__init__.py b/evdev/__init__.py index 36b330c..6aa6ef2 100644 --- a/evdev/__init__.py +++ b/evdev/__init__.py @@ -2,9 +2,8 @@ # Gather everything into a single, convenient namespace. # -------------------------------------------------------------------------- -from evdev.device import DeviceInfo, InputDevice, AbsInfo, EvdevError -from evdev.events import InputEvent, KeyEvent, RelEvent, SynEvent, AbsEvent, event_factory -from evdev.uinput import UInput, UInputError -from evdev.util import list_devices, categorize, resolve_ecodes, resolve_ecodes_dict -from evdev import ecodes -from evdev import ff +from . import ecodes, ff +from .device import AbsInfo, DeviceInfo, EvdevError, InputDevice +from .events import AbsEvent, InputEvent, KeyEvent, RelEvent, SynEvent, event_factory +from .uinput import UInput, UInputError +from .util import categorize, list_devices, resolve_ecodes, resolve_ecodes_dict diff --git a/evdev/device.py b/evdev/device.py index 758f899..7675a2d 100644 --- a/evdev/device.py +++ b/evdev/device.py @@ -1,17 +1,16 @@ # encoding: utf-8 +import collections +import contextlib import os import warnings -import contextlib -import collections -from evdev import _input, ecodes, util -from evdev.events import InputEvent +from . import _input, ecodes, util try: - from evdev.eventio_async import EventIO, EvdevError + from .eventio_async import EvdevError, EventIO except ImportError: - from evdev.eventio import EventIO, EvdevError + from .eventio import EvdevError, EventIO # -------------------------------------------------------------------------- diff --git a/evdev/ecodes.py b/evdev/ecodes.py index 759cfe7..3a6c3d0 100644 --- a/evdev/ecodes.py +++ b/evdev/ecodes.py @@ -40,8 +40,8 @@ """ from inspect import getmembers -from evdev import _ecodes +from . import _ecodes #: Mapping of names to values. ecodes = {} diff --git a/evdev/eventio.py b/evdev/eventio.py index 3335500..5478f02 100644 --- a/evdev/eventio.py +++ b/evdev/eventio.py @@ -1,10 +1,10 @@ -import os import fcntl -import select import functools +import os +import select -from evdev import _input, _uinput, ecodes, util -from evdev.events import InputEvent +from . import _input, _uinput, ecodes +from .events import InputEvent # -------------------------------------------------------------------------- diff --git a/evdev/eventio_async.py b/evdev/eventio_async.py index e89765e..fb8bcd2 100644 --- a/evdev/eventio_async.py +++ b/evdev/eventio_async.py @@ -1,10 +1,10 @@ import asyncio import select -from evdev import eventio +from . import eventio # needed for compatibility -from evdev.eventio import EvdevError +from .eventio import EvdevError class EventIO(eventio.EventIO): diff --git a/evdev/events.py b/evdev/events.py index 97f570d..9a85436 100644 --- a/evdev/events.py +++ b/evdev/events.py @@ -37,7 +37,7 @@ # event type descriptions have been taken mot-a-mot from: # http://www.kernel.org/doc/Documentation/input/event-codes.txt -from evdev.ecodes import keys, KEY, SYN, REL, ABS, EV_KEY, EV_REL, EV_ABS, EV_SYN +from .ecodes import ABS, EV_ABS, EV_KEY, EV_REL, EV_SYN, KEY, REL, SYN, keys class InputEvent: diff --git a/evdev/evtest.py b/evdev/evtest.py index b61f093..26e62ad 100644 --- a/evdev/evtest.py +++ b/evdev/evtest.py @@ -17,19 +17,14 @@ """ +import atexit +import optparse import re -import sys import select -import atexit +import sys import termios -import optparse - -try: - input = raw_input -except NameError: - pass -from evdev import ecodes, list_devices, AbsInfo, InputDevice +from . import AbsInfo, InputDevice, ecodes, list_devices def parseopt(): diff --git a/evdev/ff.py b/evdev/ff.py index edb5ff2..260c362 100644 --- a/evdev/ff.py +++ b/evdev/ff.py @@ -1,6 +1,6 @@ import ctypes -from evdev import ecodes +from . import ecodes _u8 = ctypes.c_uint8 _u16 = ctypes.c_uint16 diff --git a/evdev/uinput.py b/evdev/uinput.py index 756f83c..61de946 100644 --- a/evdev/uinput.py +++ b/evdev/uinput.py @@ -1,3 +1,4 @@ +import ctypes import os import platform import re @@ -5,11 +6,8 @@ import time from collections import defaultdict -from evdev import _uinput -from evdev import ecodes, util, device -from evdev.events import InputEvent -import evdev.ff as ff -import ctypes +from . import _uinput, device, ecodes, ff, util +from .events import InputEvent try: from evdev.eventio_async import EventIO diff --git a/evdev/util.py b/evdev/util.py index 7209f4b..59991f6 100644 --- a/evdev/util.py +++ b/evdev/util.py @@ -1,11 +1,11 @@ -import re +import collections +import glob import os +import re import stat -import glob -import collections -from evdev import ecodes -from evdev.events import event_factory +from . import ecodes +from .events import event_factory def list_devices(input_device_dir="/dev/input"): From 1818e9df54f84f6f1ed2bae87f8fe1e0c40d7682 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Mon, 20 Jan 2025 22:51:24 +0100 Subject: [PATCH 05/40] Bump required python version to 3.8 --- pyproject.toml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pyproject.toml b/pyproject.toml index 9ba60ff..37260f1 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -9,7 +9,7 @@ description = "Bindings to the Linux input handling subsystem" keywords = ["evdev", "input", "uinput"] readme = "README.md" license = {file = "LICENSE"} -requires-python = ">=3.6" +requires-python = ">=3.8" authors = [ { name="Georgi Valkov", email="georgi.t.valkov@gmail.com" }, ] From 4a0efdd252cdd35a909ab46e89f7c76ca1cf4714 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Mon, 20 Jan 2025 22:48:47 +0100 Subject: [PATCH 06/40] Generate typing stubs for evdev.ecodes --- .gitignore | 2 +- MANIFEST.in | 1 + evdev/genecodes.py | 85 ++++++++++++++++++++++++++++++++++------------ pyproject.toml | 3 ++ setup.py | 21 ++++++++---- 5 files changed, 83 insertions(+), 29 deletions(-) diff --git a/.gitignore b/.gitignore index 329a06d..6548086 100644 --- a/.gitignore +++ b/.gitignore @@ -18,7 +18,7 @@ __pycache__ evdev/*.so evdev/ecodes.c -evdev/iprops.c +evdev/ecodes.pyi docs/_build evdev/_ecodes.py evdev/_input.py diff --git a/MANIFEST.in b/MANIFEST.in index 435d617..1b5a7b6 100644 --- a/MANIFEST.in +++ b/MANIFEST.in @@ -2,3 +2,4 @@ # evdev headers of the running kernel. Refer to the 'build_ecodes' distutils # command in setup.py. exclude evdev/ecodes.c +include evdev/ecodes.pyi diff --git a/evdev/genecodes.py b/evdev/genecodes.py index ce9939e..68b7dea 100644 --- a/evdev/genecodes.py +++ b/evdev/genecodes.py @@ -2,8 +2,10 @@ Generate a Python extension module with the constants defined in linux/input.h. """ -import os, sys, re - +import getopt +import os +import re +import sys # ----------------------------------------------------------------------------- # The default header file locations to try. @@ -13,8 +15,10 @@ "/usr/include/linux/uinput.h", ] -if sys.argv[1:]: - headers = sys.argv[1:] +opts, args = getopt.getopt(sys.argv[1:], "", ["ecodes", "stubs"]) +if not opts: + print("usage: genecodes.py [--ecodes|--stubs] ") + exit(2) # ----------------------------------------------------------------------------- @@ -27,7 +31,7 @@ # ----------------------------------------------------------------------------- -template = r""" +template_ecodes = r""" #include #ifdef __FreeBSD__ #include @@ -37,7 +41,8 @@ #endif /* Automatically generated by evdev.genecodes */ -/* Generated on %s */ +/* Generated on %s */ +/* Generated from %s */ #define MODULE_NAME "_ecodes" #define MODULE_HELP "linux/input.h macros" @@ -71,25 +76,63 @@ """ -def parse_header(header): - for line in open(header): - macro = macro_regex.search(line) - if macro: - yield " PyModule_AddIntMacro(m, %s);" % macro.group(1) +template_stubs = r""" +# Automatically generated by evdev.genecodes +# Generated on %s +# Generated from %s + +# pylint: skip-file + +ecodes: dict[str, int] +keys: dict[int, str|list[str]] +bytype: dict[int, dict[int, str|list[str]]] + +KEY: dict[int, str|list[str]] +ABS: dict[int, str|list[str]] +REL: dict[int, str|list[str]] +SW: dict[int, str|list[str]] +MSC: dict[int, str|list[str]] +LED: dict[int, str|list[str]] +BTN: dict[int, str|list[str]] +REP: dict[int, str|list[str]] +SND: dict[int, str|list[str]] +ID: dict[int, str|list[str]] +EV: dict[int, str|list[str]] +BUS: dict[int, str|list[str]] +SYN: dict[int, str|list[str]] +FF_STATUS: dict[int, str|list[str]] +FF_INPUT_PROP: dict[int, str|list[str]] + +%s +""" -all_macros = [] -for header in headers: - try: - fh = open(header) - except (IOError, OSError): - continue - all_macros += parse_header(header) +def parse_headers(headers=headers): + for header in headers: + try: + fh = open(header) + except (IOError, OSError): + continue + for line in fh: + macro = macro_regex.search(line) + if macro: + yield macro.group(1) + + +all_macros = list(parse_headers()) if not all_macros: print("no input macros found in: %s" % " ".join(headers), file=sys.stderr) sys.exit(1) - -macros = os.linesep.join(all_macros) -print(template % (uname, macros)) +# pylint: disable=possibly-used-before-assignment, used-before-assignment +if ("--ecodes", "") in opts: + body = (" PyModule_AddIntMacro(m, %s);" % macro for macro in all_macros) + template = template_ecodes +elif ("--stubs", "") in opts: + body = ("%s: int" % macro for macro in all_macros) + template = template_stubs + +body = os.linesep.join(body) +text = template % (uname, headers, body) +print(text.strip()) diff --git a/pyproject.toml b/pyproject.toml index 37260f1..3175a51 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -32,6 +32,9 @@ classifiers = [ [tool.setuptools] packages = ["evdev"] +[tool.setuptools.data-files] +"data" = ["evdev/*.pyi"] + [tool.ruff] line-length = 120 diff --git a/setup.py b/setup.py index 6781527..0990554 100755 --- a/setup.py +++ b/setup.py @@ -8,7 +8,8 @@ curdir = Path(__file__).resolve().parent -ecodes_path = curdir / "evdev/ecodes.c" +ecodes_c_path = curdir / "evdev/ecodes.c" +ecodes_pyi_path = curdir / "evdev/ecodes.pyi" def create_ecodes(headers=None): @@ -58,9 +59,14 @@ def create_ecodes(headers=None): from subprocess import run - print("writing %s (using %s)" % (ecodes_path, " ".join(headers))) - with ecodes_path.open("w") as fh: - cmd = [sys.executable, "evdev/genecodes.py", *headers] + print("writing %s (using %s)" % (ecodes_c_path, " ".join(headers))) + with ecodes_c_path.open("w") as fh: + cmd = [sys.executable, "evdev/genecodes.py", "--ecodes", *headers] + run(cmd, check=True, stdout=fh) + + print("writing %s (using %s)" % (ecodes_pyi_path, " ".join(headers))) + with ecodes_pyi_path.open("w") as fh: + cmd = [sys.executable, "evdev/genecodes.py", "--stubs", *headers] run(cmd, check=True, stdout=fh) @@ -84,9 +90,10 @@ def run(self): class build_ext(_build_ext.build_ext): def has_ecodes(self): - if ecodes_path.exists(): - print("ecodes.c already exists ... skipping build_ecodes") - return not ecodes_path.exists() + if ecodes_c_path.exists() and ecodes_pyi_path.exists(): + print("ecodes.c and ecodes.pyi already exist ... skipping build_ecodes") + return False + return True def run(self): for cmd_name in self.get_sub_commands(): From abd286e3d8880d99556f37f6e349ce6cecaaf267 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Mon, 20 Jan 2025 23:45:41 +0100 Subject: [PATCH 07/40] Pylint fixes --- .github/workflows/install.yaml | 2 +- .github/workflows/lint.yml | 2 +- evdev/events.py | 1 + pyproject.toml | 9 +++++++++ 4 files changed, 12 insertions(+), 2 deletions(-) diff --git a/.github/workflows/install.yaml b/.github/workflows/install.yaml index f959de2..87502ad 100644 --- a/.github/workflows/install.yaml +++ b/.github/workflows/install.yaml @@ -11,7 +11,7 @@ jobs: fail-fast: false matrix: os: [ubuntu-latest] - python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"] + python-version: ["3.8", "3.9", "3.10", "3.11", "3.12", "3.13"] include: - os: ubuntu-latest python-version: "3.8" diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index d499462..e293976 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -24,4 +24,4 @@ jobs: run: | python -m pip install pylint setuptools python setup.py build - python -m pylint --disable=no-member --verbose -E build/lib*/evdev + python -m pylint --verbose -E build/lib*/evdev diff --git a/evdev/events.py b/evdev/events.py index 9a85436..a4f817d 100644 --- a/evdev/events.py +++ b/evdev/events.py @@ -37,6 +37,7 @@ # event type descriptions have been taken mot-a-mot from: # http://www.kernel.org/doc/Documentation/input/event-codes.txt +# pylint: disable=no-name-in-module from .ecodes import ABS, EV_ABS, EV_KEY, EV_REL, EV_SYN, KEY, REL, SYN, keys diff --git a/pyproject.toml b/pyproject.toml index 3175a51..5f56454 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -52,3 +52,12 @@ filename = "pyproject.toml" [[tool.bumpversion.files]] filename = "docs/conf.py" + +[tool.pylint.'MESSAGES CONTROL'] +disable = """ + no-member, +""" + +[tool.pylint.typecheck] +generated-members = ["evdev.ecodes.*"] +ignored-modules= ["evdev._*"] From 83f9360948534efd4bec82d5db213b49afd9cf15 Mon Sep 17 00:00:00 2001 From: Tobi <28510156+sezanzeb@users.noreply.github.com> Date: Tue, 21 Jan 2025 12:11:15 +0100 Subject: [PATCH 08/40] Small character device verification cleanup (#229) --- evdev/uinput.py | 8 +++----- tests/test_uinput.py | 17 +++++++++++++++-- 2 files changed, 18 insertions(+), 7 deletions(-) diff --git a/evdev/uinput.py b/evdev/uinput.py index 61de946..9567374 100644 --- a/evdev/uinput.py +++ b/evdev/uinput.py @@ -266,13 +266,11 @@ def _verify(self): Verify that an uinput device exists and is readable and writable by the current process. """ - try: m = os.stat(self.devnode)[stat.ST_MODE] - if not stat.S_ISCHR(m): - raise OSError - except (IndexError, OSError): - msg = '"{}" does not exist or is not a character device file ' "- verify that the uinput module is loaded" + assert stat.S_ISCHR(m) + except (IndexError, OSError, AssertionError): + msg = '"{}" does not exist or is not a character device file - verify that the uinput module is loaded' raise UInputError(msg.format(self.devnode)) if not os.access(self.devnode, os.W_OK): diff --git a/tests/test_uinput.py b/tests/test_uinput.py index dcd09e0..666361f 100644 --- a/tests/test_uinput.py +++ b/tests/test_uinput.py @@ -1,4 +1,5 @@ # encoding: utf-8 +import os import stat from select import select from unittest.mock import patch @@ -119,6 +120,18 @@ def test_write(c): @patch.object(stat, 'S_ISCHR', return_value=False) -def test_not_a_character_device(c): - with pytest.raises(UInputError): +def test_not_a_character_device(ischr_mock, c): + with pytest.raises(UInputError, match='not a character device file'): + uinput.UInput(**c) + +@patch.object(stat, 'S_ISCHR', return_value=True) +@patch.object(os, 'stat', side_effect=OSError()) +def test_not_a_character_device_2(stat_mock, ischr_mock, c): + with pytest.raises(UInputError, match='not a character device file'): + uinput.UInput(**c) + +@patch.object(stat, 'S_ISCHR', return_value=True) +@patch.object(os, 'stat', return_value=[]) +def test_not_a_character_device_3(stat_mock, ischr_mock, c): + with pytest.raises(UInputError, match='not a character device file'): uinput.UInput(**c) From 4ca0a8b41915650e10fbdd4114519c9183406940 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Wed, 22 Jan 2025 00:43:21 +0100 Subject: [PATCH 09/40] Generate ecodes.py at build time - The existing ecodes.py is renamed to ecodes_runtime.py. - An ecodes.py is generated at build time (in build_ext) with the genecodes_py.py script, after the extension modules are built. The script essentially does a repr() on vars(ecodes_runtime) and adds type annotations. - If something goes wrong in the process of generating ecodes.py, ecodes_runtime.py is copied to ecodes.py. - Stop generating ecodes.pyi as the generated ecodes.py is fully annotated. --- .gitignore | 1 + MANIFEST.in | 2 +- docs/changelog.rst | 11 +++ evdev/ecodes.py | 105 +------------------------ evdev/ecodes_runtime.py | 102 ++++++++++++++++++++++++ evdev/{genecodes.py => genecodes_c.py} | 0 evdev/genecodes_py.py | 53 +++++++++++++ pyproject.toml | 3 - setup.py | 30 ++++--- 9 files changed, 190 insertions(+), 117 deletions(-) create mode 100644 evdev/ecodes_runtime.py rename evdev/{genecodes.py => genecodes_c.py} (100%) create mode 100644 evdev/genecodes_py.py diff --git a/.gitignore b/.gitignore index 6548086..3e244aa 100644 --- a/.gitignore +++ b/.gitignore @@ -15,6 +15,7 @@ TAGS .#* __pycache__ .pytest_cache +.ruff_cache evdev/*.so evdev/ecodes.c diff --git a/MANIFEST.in b/MANIFEST.in index 1b5a7b6..bcbbd6c 100644 --- a/MANIFEST.in +++ b/MANIFEST.in @@ -2,4 +2,4 @@ # evdev headers of the running kernel. Refer to the 'build_ecodes' distutils # command in setup.py. exclude evdev/ecodes.c -include evdev/ecodes.pyi +include evdev/ecodes.py diff --git a/docs/changelog.rst b/docs/changelog.rst index c14026a..81ff3d4 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -7,6 +7,17 @@ Changelog - Binary wheels are now provided by the `evdev-binary http://pypi.python.org/pypi/evdev-binary`_ package. The package is compiled on manylinux_2_28 against kernel 4.18. +- The ``evdev.ecodes`` module is now generated at install time and contains only constants. This allows type + checking and introspection of the ``evdev.ecodes`` module, without having to execute it first. The old + module is available as ``evdev.ecodes_runtime``. In case generation of the static ``ecodes.py`` fails, the + install process falls back to using ``ecodes_runtime.py`` as ``ecodes.py``. + +- Minimum Python version raised to Python 3.8. + +- Fix keyboard delay and repeat being swapped. + +- Move `syn()` convenience method from `InputDevice` to `EventIO`. + 1.7.1 (May 8, 2024) ==================== diff --git a/evdev/ecodes.py b/evdev/ecodes.py index 3a6c3d0..a19dcba 100644 --- a/evdev/ecodes.py +++ b/evdev/ecodes.py @@ -1,102 +1,5 @@ -# pylint: disable=undefined-variable -""" -This modules exposes the integer constants defined in ``linux/input.h`` and -``linux/input-event-codes.h``. +# When installed, this module is replaced by an ecodes.py generated at +# build time by genecodes_py.py (see build_ext in setup.py). -Exposed constants:: - - KEY, ABS, REL, SW, MSC, LED, BTN, REP, SND, ID, EV, - BUS, SYN, FF, FF_STATUS, INPUT_PROP - -This module also provides reverse and forward mappings of the names and values -of the above mentioned constants:: - - >>> evdev.ecodes.KEY_A - 30 - - >>> evdev.ecodes.ecodes['KEY_A'] - 30 - - >>> evdev.ecodes.KEY[30] - 'KEY_A' - - >>> evdev.ecodes.REL[0] - 'REL_X' - - >>> evdev.ecodes.EV[evdev.ecodes.EV_KEY] - 'EV_KEY' - - >>> evdev.ecodes.bytype[evdev.ecodes.EV_REL][0] - 'REL_X' - -Keep in mind that values in reverse mappings may point to one or more event -codes. For example:: - - >>> evdev.ecodes.FF[80] - ['FF_EFFECT_MIN', 'FF_RUMBLE'] - - >>> evdev.ecodes.FF[81] - 'FF_PERIODIC' -""" - -from inspect import getmembers - -from . import _ecodes - -#: Mapping of names to values. -ecodes = {} - -prefixes = "KEY ABS REL SW MSC LED BTN REP SND ID EV BUS SYN FF_STATUS FF INPUT_PROP" -prev_prefix = "" -g = globals() - -# eg. code: 'REL_Z', val: 2 -for code, val in getmembers(_ecodes): - for prefix in prefixes.split(): # eg. 'REL' - if code.startswith(prefix): - ecodes[code] = val - # FF_STATUS codes should not appear in the FF reverse mapping - if not code.startswith(prev_prefix): - d = g.setdefault(prefix, {}) - # codes that share the same value will be added to a list. eg: - # >>> ecodes.FF_STATUS - # {0: 'FF_STATUS_STOPPED', 1: ['FF_STATUS_MAX', 'FF_STATUS_PLAYING']} - if val in d: - if isinstance(d[val], list): - d[val].append(code) - else: - d[val] = [d[val], code] - else: - d[val] = code - - prev_prefix = prefix - -#: Keys are a combination of all BTN and KEY codes. -keys = {} -keys.update(BTN) -keys.update(KEY) - -# make keys safe to use for the default list of uinput device -# capabilities -del keys[_ecodes.KEY_MAX] -del keys[_ecodes.KEY_CNT] - -#: Mapping of event types to other value/name mappings. -bytype = { - _ecodes.EV_KEY: keys, - _ecodes.EV_ABS: ABS, - _ecodes.EV_REL: REL, - _ecodes.EV_SW: SW, - _ecodes.EV_MSC: MSC, - _ecodes.EV_LED: LED, - _ecodes.EV_REP: REP, - _ecodes.EV_SND: SND, - _ecodes.EV_SYN: SYN, - _ecodes.EV_FF: FF, - _ecodes.EV_FF_STATUS: FF_STATUS, -} - -from evdev._ecodes import * - -# cheaper than whitelisting in an __all__ -del code, val, prefix, getmembers, g, d, prefixes, prev_prefix +# This stub exists to make development of evdev itself more convenient. +from . ecodes_runtime import * diff --git a/evdev/ecodes_runtime.py b/evdev/ecodes_runtime.py new file mode 100644 index 0000000..3a6c3d0 --- /dev/null +++ b/evdev/ecodes_runtime.py @@ -0,0 +1,102 @@ +# pylint: disable=undefined-variable +""" +This modules exposes the integer constants defined in ``linux/input.h`` and +``linux/input-event-codes.h``. + +Exposed constants:: + + KEY, ABS, REL, SW, MSC, LED, BTN, REP, SND, ID, EV, + BUS, SYN, FF, FF_STATUS, INPUT_PROP + +This module also provides reverse and forward mappings of the names and values +of the above mentioned constants:: + + >>> evdev.ecodes.KEY_A + 30 + + >>> evdev.ecodes.ecodes['KEY_A'] + 30 + + >>> evdev.ecodes.KEY[30] + 'KEY_A' + + >>> evdev.ecodes.REL[0] + 'REL_X' + + >>> evdev.ecodes.EV[evdev.ecodes.EV_KEY] + 'EV_KEY' + + >>> evdev.ecodes.bytype[evdev.ecodes.EV_REL][0] + 'REL_X' + +Keep in mind that values in reverse mappings may point to one or more event +codes. For example:: + + >>> evdev.ecodes.FF[80] + ['FF_EFFECT_MIN', 'FF_RUMBLE'] + + >>> evdev.ecodes.FF[81] + 'FF_PERIODIC' +""" + +from inspect import getmembers + +from . import _ecodes + +#: Mapping of names to values. +ecodes = {} + +prefixes = "KEY ABS REL SW MSC LED BTN REP SND ID EV BUS SYN FF_STATUS FF INPUT_PROP" +prev_prefix = "" +g = globals() + +# eg. code: 'REL_Z', val: 2 +for code, val in getmembers(_ecodes): + for prefix in prefixes.split(): # eg. 'REL' + if code.startswith(prefix): + ecodes[code] = val + # FF_STATUS codes should not appear in the FF reverse mapping + if not code.startswith(prev_prefix): + d = g.setdefault(prefix, {}) + # codes that share the same value will be added to a list. eg: + # >>> ecodes.FF_STATUS + # {0: 'FF_STATUS_STOPPED', 1: ['FF_STATUS_MAX', 'FF_STATUS_PLAYING']} + if val in d: + if isinstance(d[val], list): + d[val].append(code) + else: + d[val] = [d[val], code] + else: + d[val] = code + + prev_prefix = prefix + +#: Keys are a combination of all BTN and KEY codes. +keys = {} +keys.update(BTN) +keys.update(KEY) + +# make keys safe to use for the default list of uinput device +# capabilities +del keys[_ecodes.KEY_MAX] +del keys[_ecodes.KEY_CNT] + +#: Mapping of event types to other value/name mappings. +bytype = { + _ecodes.EV_KEY: keys, + _ecodes.EV_ABS: ABS, + _ecodes.EV_REL: REL, + _ecodes.EV_SW: SW, + _ecodes.EV_MSC: MSC, + _ecodes.EV_LED: LED, + _ecodes.EV_REP: REP, + _ecodes.EV_SND: SND, + _ecodes.EV_SYN: SYN, + _ecodes.EV_FF: FF, + _ecodes.EV_FF_STATUS: FF_STATUS, +} + +from evdev._ecodes import * + +# cheaper than whitelisting in an __all__ +del code, val, prefix, getmembers, g, d, prefixes, prev_prefix diff --git a/evdev/genecodes.py b/evdev/genecodes_c.py similarity index 100% rename from evdev/genecodes.py rename to evdev/genecodes_c.py diff --git a/evdev/genecodes_py.py b/evdev/genecodes_py.py new file mode 100644 index 0000000..bd97553 --- /dev/null +++ b/evdev/genecodes_py.py @@ -0,0 +1,53 @@ +import sys +from unittest import mock +from pprint import PrettyPrinter + +sys.modules["evdev.ecodes"] = mock.Mock() +from evdev import ecodes_runtime as ecodes + +pprint = PrettyPrinter(indent=2, sort_dicts=True, width=120).pprint + + +print("# Automatically generated by evdev.genecodes_py") +print() +print('"""') +print(ecodes.__doc__.strip()) +print('"""') + +print() +print("from typing import Final, Dict, List, Union") +print() + +for name, value in ecodes.ecodes.items(): + print(f"{name}: Final[int] = {value}") +print() + +entries = [ + ("ecodes", "Dict[str, int]", "#: Mapping of names to values."), + ("bytype", "Dict[int, Dict[int, Union[str, List[str]]]]", "#: Mapping of event types to other value/name mappings."), + ("keys", "Dict[int, Union[str, List[str]]]", "#: Keys are a combination of all BTN and KEY codes."), + ("KEY", "Dict[int, Union[str, List[str]]]", None), + ("ABS", "Dict[int, Union[str, List[str]]]", None), + ("REL", "Dict[int, Union[str, List[str]]]", None), + ("SW", "Dict[int, Union[str, List[str]]]", None), + ("MSC", "Dict[int, Union[str, List[str]]]", None), + ("LED", "Dict[int, Union[str, List[str]]]", None), + ("BTN", "Dict[int, Union[str, List[str]]]", None), + ("REP", "Dict[int, Union[str, List[str]]]", None), + ("SND", "Dict[int, Union[str, List[str]]]", None), + ("ID", "Dict[int, Union[str, List[str]]]", None), + ("EV", "Dict[int, Union[str, List[str]]]", None), + ("BUS", "Dict[int, Union[str, List[str]]]", None), + ("SYN", "Dict[int, Union[str, List[str]]]", None), + ("FF", "Dict[int, Union[str, List[str]]]", None), + ("FF_STATUS", "Dict[int, Union[str, List[str]]]", None), + ("INPUT_PROP", "Dict[int, Union[str, List[str]]]", None) +] + +for key, annotation, doc in entries: + if doc: + print(doc) + + print(f"{key}: {annotation} = ", end="") + pprint(getattr(ecodes, key)) + print() diff --git a/pyproject.toml b/pyproject.toml index 5f56454..e7ea393 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -32,9 +32,6 @@ classifiers = [ [tool.setuptools] packages = ["evdev"] -[tool.setuptools.data-files] -"data" = ["evdev/*.pyi"] - [tool.ruff] line-length = 120 diff --git a/setup.py b/setup.py index 0990554..e6e0c1d 100755 --- a/setup.py +++ b/setup.py @@ -1,7 +1,9 @@ import os import sys +import shutil import textwrap from pathlib import Path +from subprocess import run from setuptools import setup, Extension, Command from setuptools.command import build_ext as _build_ext @@ -9,7 +11,6 @@ curdir = Path(__file__).resolve().parent ecodes_c_path = curdir / "evdev/ecodes.c" -ecodes_pyi_path = curdir / "evdev/ecodes.pyi" def create_ecodes(headers=None): @@ -49,7 +50,7 @@ def create_ecodes(headers=None): build_ext --include-dirs path/ \\ install - If you prefer to avoid building this package from source, then please consider + If you want to avoid building this package from source, then please consider installing the `evdev-binary` package instead. Keep in mind that it may not be fully compatible with, or support all the features of your current kernel. """ @@ -57,16 +58,9 @@ def create_ecodes(headers=None): sys.stderr.write(textwrap.dedent(msg)) sys.exit(1) - from subprocess import run - print("writing %s (using %s)" % (ecodes_c_path, " ".join(headers))) with ecodes_c_path.open("w") as fh: - cmd = [sys.executable, "evdev/genecodes.py", "--ecodes", *headers] - run(cmd, check=True, stdout=fh) - - print("writing %s (using %s)" % (ecodes_pyi_path, " ".join(headers))) - with ecodes_pyi_path.open("w") as fh: - cmd = [sys.executable, "evdev/genecodes.py", "--stubs", *headers] + cmd = [sys.executable, "evdev/genecodes_c.py", "--ecodes", *headers] run(cmd, check=True, stdout=fh) @@ -90,15 +84,27 @@ def run(self): class build_ext(_build_ext.build_ext): def has_ecodes(self): - if ecodes_c_path.exists() and ecodes_pyi_path.exists(): - print("ecodes.c and ecodes.pyi already exist ... skipping build_ecodes") + if ecodes_c_path.exists(): + print("ecodes.c already exists ... skipping build_ecodes") return False return True + def generate_ecodes_py(self): + ecodes_py = Path(self.build_lib) / "evdev/ecodes.py" + print(f"writing {ecodes_py}") + with ecodes_py.open("w") as fh: + cmd = [sys.executable, "-B", "evdev/genecodes_py.py"] + res = run(cmd, env={"PYTHONPATH": self.build_lib}, stdout=fh) + + if res.returncode != 0: + print(f"failed to generate static {ecodes_py} - will use ecodes_runtime.py") + shutil.copy("evdev/ecodes_runtime.py", ecodes_py) + def run(self): for cmd_name in self.get_sub_commands(): self.run_command(cmd_name) _build_ext.build_ext.run(self) + self.generate_ecodes_py() sub_commands = [("build_ecodes", has_ecodes)] + _build_ext.build_ext.sub_commands From dfd45df12abcb6b55f3f19ec1fb100821284a089 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sat, 25 Jan 2025 17:36:10 +0100 Subject: [PATCH 10/40] ecodes mappings that point to more than one value are now tuples --- docs/changelog.rst | 7 ++++++- evdev/ecodes_runtime.py | 17 +++++++++++++---- evdev/genecodes_py.py | 38 +++++++++++++++++++------------------- 3 files changed, 38 insertions(+), 24 deletions(-) diff --git a/docs/changelog.rst b/docs/changelog.rst index 81ff3d4..15ba749 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -1,7 +1,7 @@ Changelog --------- -1.8.0 (Unreleased) +1.8.0 (Jan 25, 2025) ================== - Binary wheels are now provided by the `evdev-binary http://pypi.python.org/pypi/evdev-binary`_ package. @@ -12,6 +12,11 @@ Changelog module is available as ``evdev.ecodes_runtime``. In case generation of the static ``ecodes.py`` fails, the install process falls back to using ``ecodes_runtime.py`` as ``ecodes.py``. +- Reverse mappings in ``evdev.ecodes`` that point to more than one value are now tuples and not lists. For example:: + + >>> ecodes.KEY[153] + 153: ('KEY_DIRECTION', 'KEY_ROTATE_DISPLAY'), + - Minimum Python version raised to Python 3.8. - Fix keyboard delay and repeat being swapped. diff --git a/evdev/ecodes_runtime.py b/evdev/ecodes_runtime.py index 3a6c3d0..d6c8b2a 100644 --- a/evdev/ecodes_runtime.py +++ b/evdev/ecodes_runtime.py @@ -33,7 +33,7 @@ codes. For example:: >>> evdev.ecodes.FF[80] - ['FF_EFFECT_MIN', 'FF_RUMBLE'] + ('FF_EFFECT_MIN', 'FF_RUMBLE') >>> evdev.ecodes.FF[81] 'FF_PERIODIC' @@ -46,13 +46,13 @@ #: Mapping of names to values. ecodes = {} -prefixes = "KEY ABS REL SW MSC LED BTN REP SND ID EV BUS SYN FF_STATUS FF INPUT_PROP" +prefixes = "KEY ABS REL SW MSC LED BTN REP SND ID EV BUS SYN FF_STATUS FF INPUT_PROP".split() prev_prefix = "" g = globals() # eg. code: 'REL_Z', val: 2 for code, val in getmembers(_ecodes): - for prefix in prefixes.split(): # eg. 'REL' + for prefix in prefixes: # eg. 'REL' if code.startswith(prefix): ecodes[code] = val # FF_STATUS codes should not appear in the FF reverse mapping @@ -71,6 +71,15 @@ prev_prefix = prefix + +# Convert lists to tuples. +k, v = None, None +for prefix in prefixes: + for k, v in g[prefix].items(): + if isinstance(v, list): + g[prefix][k] = tuple(v) + + #: Keys are a combination of all BTN and KEY codes. keys = {} keys.update(BTN) @@ -99,4 +108,4 @@ from evdev._ecodes import * # cheaper than whitelisting in an __all__ -del code, val, prefix, getmembers, g, d, prefixes, prev_prefix +del code, val, prefix, getmembers, g, d, k, v, prefixes, prev_prefix diff --git a/evdev/genecodes_py.py b/evdev/genecodes_py.py index bd97553..1afbc34 100644 --- a/evdev/genecodes_py.py +++ b/evdev/genecodes_py.py @@ -15,7 +15,7 @@ print('"""') print() -print("from typing import Final, Dict, List, Union") +print("from typing import Final, Dict, Tuple, Union") print() for name, value in ecodes.ecodes.items(): @@ -24,24 +24,24 @@ entries = [ ("ecodes", "Dict[str, int]", "#: Mapping of names to values."), - ("bytype", "Dict[int, Dict[int, Union[str, List[str]]]]", "#: Mapping of event types to other value/name mappings."), - ("keys", "Dict[int, Union[str, List[str]]]", "#: Keys are a combination of all BTN and KEY codes."), - ("KEY", "Dict[int, Union[str, List[str]]]", None), - ("ABS", "Dict[int, Union[str, List[str]]]", None), - ("REL", "Dict[int, Union[str, List[str]]]", None), - ("SW", "Dict[int, Union[str, List[str]]]", None), - ("MSC", "Dict[int, Union[str, List[str]]]", None), - ("LED", "Dict[int, Union[str, List[str]]]", None), - ("BTN", "Dict[int, Union[str, List[str]]]", None), - ("REP", "Dict[int, Union[str, List[str]]]", None), - ("SND", "Dict[int, Union[str, List[str]]]", None), - ("ID", "Dict[int, Union[str, List[str]]]", None), - ("EV", "Dict[int, Union[str, List[str]]]", None), - ("BUS", "Dict[int, Union[str, List[str]]]", None), - ("SYN", "Dict[int, Union[str, List[str]]]", None), - ("FF", "Dict[int, Union[str, List[str]]]", None), - ("FF_STATUS", "Dict[int, Union[str, List[str]]]", None), - ("INPUT_PROP", "Dict[int, Union[str, List[str]]]", None) + ("bytype", "Dict[int, Dict[int, Union[str, Tuple[str]]]]", "#: Mapping of event types to other value/name mappings."), + ("keys", "Dict[int, Union[str, Tuple[str]]]", "#: Keys are a combination of all BTN and KEY codes."), + ("KEY", "Dict[int, Union[str, Tuple[str]]]", None), + ("ABS", "Dict[int, Union[str, Tuple[str]]]", None), + ("REL", "Dict[int, Union[str, Tuple[str]]]", None), + ("SW", "Dict[int, Union[str, Tuple[str]]]", None), + ("MSC", "Dict[int, Union[str, Tuple[str]]]", None), + ("LED", "Dict[int, Union[str, Tuple[str]]]", None), + ("BTN", "Dict[int, Union[str, Tuple[str]]]", None), + ("REP", "Dict[int, Union[str, Tuple[str]]]", None), + ("SND", "Dict[int, Union[str, Tuple[str]]]", None), + ("ID", "Dict[int, Union[str, Tuple[str]]]", None), + ("EV", "Dict[int, Union[str, Tuple[str]]]", None), + ("BUS", "Dict[int, Union[str, Tuple[str]]]", None), + ("SYN", "Dict[int, Union[str, Tuple[str]]]", None), + ("FF", "Dict[int, Union[str, Tuple[str]]]", None), + ("FF_STATUS", "Dict[int, Union[str, Tuple[str]]]", None), + ("INPUT_PROP", "Dict[int, Union[str, Tuple[str]]]", None) ] for key, annotation, doc in entries: From 2e3b843f37f79a9404da4541bbf9cf96dc5a4d57 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sat, 25 Jan 2025 17:37:33 +0100 Subject: [PATCH 11/40] =?UTF-8?q?Bump=20version:=201.7.1=20=E2=86=92=201.8?= =?UTF-8?q?.0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- LICENSE | 2 +- docs/conf.py | 2 +- pyproject.toml | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/LICENSE b/LICENSE index 5600871..8482b07 100644 --- a/LICENSE +++ b/LICENSE @@ -1,4 +1,4 @@ -Copyright (c) 2012-2023 Georgi Valkov. All rights reserved. +Copyright (c) 2012-2025 Georgi Valkov. All rights reserved. Redistribution and use in source and binary forms, with or without modification, are permitted provided that the following conditions are diff --git a/docs/conf.py b/docs/conf.py index bf03b42..7af99b9 100644 --- a/docs/conf.py +++ b/docs/conf.py @@ -67,7 +67,7 @@ # built documents. # # The full version, including alpha/beta/rc tags. -release = "1.7.1" +release = "1.8.0" # The short X.Y version. version = release diff --git a/pyproject.toml b/pyproject.toml index e7ea393..e5e5d00 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta" [project] name = "evdev" -version = "1.7.1" +version = "1.8.0" description = "Bindings to the Linux input handling subsystem" keywords = ["evdev", "input", "uinput"] readme = "README.md" @@ -39,7 +39,7 @@ line-length = 120 ignore = ["E265", "E241", "F403", "F401", "E401", "E731"] [tool.bumpversion] -current_version = "1.7.1" +current_version = "1.8.0" commit = true tag = true allow_dirty = true From 27eb2ff11bb6b41fa0cfcff4f80d6c26d4b65742 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sat, 25 Jan 2025 18:04:39 +0100 Subject: [PATCH 12/40] Fix tests --- tests/test_util.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/test_util.py b/tests/test_util.py index 5a979df..7112927 100644 --- a/tests/test_util.py +++ b/tests/test_util.py @@ -6,7 +6,7 @@ def test_match_ecodes_a(): assert res == {1: [372, 418, 419, 420]} assert dict(util.resolve_ecodes_dict(res)) == { ("EV_KEY", 1): [ - (["KEY_FULL_SCREEN", "KEY_ZOOM"], 372), + (("KEY_FULL_SCREEN", "KEY_ZOOM"), 372), ("KEY_ZOOMIN", 418), ("KEY_ZOOMOUT", 419), ("KEY_ZOOMRESET", 420), From 4b8fa71d3c0123916138d19729e13caee03ab47f Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sat, 25 Jan 2025 21:23:11 +0100 Subject: [PATCH 13/40] Fix docs --- docs/changelog.rst | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/docs/changelog.rst b/docs/changelog.rst index 15ba749..92f4e23 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -4,7 +4,7 @@ Changelog 1.8.0 (Jan 25, 2025) ================== -- Binary wheels are now provided by the `evdev-binary http://pypi.python.org/pypi/evdev-binary`_ package. +- Binary wheels are now provided by the `evdev-binary `_ package. The package is compiled on manylinux_2_28 against kernel 4.18. - The ``evdev.ecodes`` module is now generated at install time and contains only constants. This allows type @@ -12,16 +12,16 @@ Changelog module is available as ``evdev.ecodes_runtime``. In case generation of the static ``ecodes.py`` fails, the install process falls back to using ``ecodes_runtime.py`` as ``ecodes.py``. -- Reverse mappings in ``evdev.ecodes`` that point to more than one value are now tuples and not lists. For example:: +- Reverse mappings in ``evdev.ecodes`` that point to more than one value are now tuples instead of lists. For example:: >>> ecodes.KEY[153] - 153: ('KEY_DIRECTION', 'KEY_ROTATE_DISPLAY'), + ('KEY_DIRECTION', 'KEY_ROTATE_DISPLAY') -- Minimum Python version raised to Python 3.8. +- Raise the minimum supported Python version to 3.8. -- Fix keyboard delay and repeat being swapped. +- Fix keyboard delay and repeat being swapped (#227). -- Move `syn()` convenience method from `InputDevice` to `EventIO`. +- Move the ``syn()`` convenience method from ``InputDevice`` to ``EventIO`` (#224). 1.7.1 (May 8, 2024) @@ -41,7 +41,7 @@ Changelog - Add the uniq address to the string representation of ``InputDevice``. -- Improved method for finding the device node corresponding to a uinput device (`#206 https://github.com/gvalkov/python-evdev/pull/206`_). +- Improved method for finding the device node corresponding to a uinput device (`#206 `_). - Repository TLC (reformatted with ruff, fixed linting warnings, moved packaging metadata to ``pyproject.toml`` etc.). From 3ff9816e08be95b331ea9dadc18fc17f1e04e272 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 14:19:12 +0100 Subject: [PATCH 14/40] Fix ecodes.c generation Header files passed to genecodes_c.py were ignored after commit 4a0efdd. --- evdev/genecodes_c.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/evdev/genecodes_c.py b/evdev/genecodes_c.py index 68b7dea..dd3ee91 100644 --- a/evdev/genecodes_c.py +++ b/evdev/genecodes_c.py @@ -20,6 +20,9 @@ print("usage: genecodes.py [--ecodes|--stubs] ") exit(2) +if args: + headers = args + # ----------------------------------------------------------------------------- macro_regex = r"#define +((?:KEY|ABS|REL|SW|MSC|LED|BTN|REP|SND|ID|EV|BUS|SYN|FF|UI_FF|INPUT_PROP)_\w+)" From 61beda72e7b101e270f914d5f1d633730e60d083 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 17:25:16 +0100 Subject: [PATCH 15/40] Move from flat-layout to src-layout --- .gitignore | 12 ++++++------ MANIFEST.in | 4 ++-- docs/conf.py | 10 ++++------ pyproject.toml | 3 --- setup.py | 14 +++++++------- {evdev => src/evdev}/__init__.py | 0 {evdev => src/evdev}/device.py | 0 {evdev => src/evdev}/ecodes.py | 0 {evdev => src/evdev}/ecodes_runtime.py | 0 {evdev => src/evdev}/eventio.py | 0 {evdev => src/evdev}/eventio_async.py | 0 {evdev => src/evdev}/events.py | 0 {evdev => src/evdev}/evtest.py | 0 {evdev => src/evdev}/ff.py | 0 {evdev => src/evdev}/genecodes_c.py | 0 {evdev => src/evdev}/genecodes_py.py | 0 {evdev => src/evdev}/input.c | 0 {evdev => src/evdev}/uinput.c | 0 {evdev => src/evdev}/uinput.py | 0 {evdev => src/evdev}/util.py | 0 20 files changed, 19 insertions(+), 24 deletions(-) rename {evdev => src/evdev}/__init__.py (100%) rename {evdev => src/evdev}/device.py (100%) rename {evdev => src/evdev}/ecodes.py (100%) rename {evdev => src/evdev}/ecodes_runtime.py (100%) rename {evdev => src/evdev}/eventio.py (100%) rename {evdev => src/evdev}/eventio_async.py (100%) rename {evdev => src/evdev}/events.py (100%) rename {evdev => src/evdev}/evtest.py (100%) rename {evdev => src/evdev}/ff.py (100%) rename {evdev => src/evdev}/genecodes_c.py (100%) rename {evdev => src/evdev}/genecodes_py.py (100%) rename {evdev => src/evdev}/input.c (100%) rename {evdev => src/evdev}/uinput.c (100%) rename {evdev => src/evdev}/uinput.py (100%) rename {evdev => src/evdev}/util.py (100%) diff --git a/.gitignore b/.gitignore index 3e244aa..557f265 100644 --- a/.gitignore +++ b/.gitignore @@ -17,10 +17,10 @@ __pycache__ .pytest_cache .ruff_cache -evdev/*.so -evdev/ecodes.c -evdev/ecodes.pyi +src/evdev/*.so +src/evdev/ecodes.c +src/evdev/ecodes.pyi docs/_build -evdev/_ecodes.py -evdev/_input.py -evdev/_uinput.py +src/evdev/_ecodes.py +src/evdev/_input.py +src/evdev/_uinput.py diff --git a/MANIFEST.in b/MANIFEST.in index bcbbd6c..be2be3d 100644 --- a/MANIFEST.in +++ b/MANIFEST.in @@ -1,5 +1,5 @@ # The _ecodes extension module source file needs to be generated against the # evdev headers of the running kernel. Refer to the 'build_ecodes' distutils # command in setup.py. -exclude evdev/ecodes.c -include evdev/ecodes.py +exclude src/evdev/ecodes.c +include src/evdev/ecodes.py diff --git a/docs/conf.py b/docs/conf.py index 7af99b9..53b5206 100644 --- a/docs/conf.py +++ b/docs/conf.py @@ -1,5 +1,3 @@ -# -*- coding: utf-8 -*- - import os import sys import sphinx_rtd_theme @@ -13,7 +11,7 @@ # Trick autodoc into running without having built the extension modules. if on_rtd: - with open("../evdev/_ecodes.py", "w") as fh: + with open("../src/evdev/_ecodes.py", "w") as fh: fh.write( """ KEY = ABS = REL = SW = MSC = LED = REP = SND = SYN = FF = FF_STATUS = BTN_A = KEY_A = 1 @@ -22,9 +20,9 @@ KEY_MAX, KEY_CNT = 1, 2""" ) - with open("../evdev/_input.py", "w"): + with open("../src/evdev/_input.py", "w"): pass - with open("../evdev/_uinput.py", "w"): + with open("../src/evdev/_uinput.py", "w"): pass @@ -60,7 +58,7 @@ # General information about the project. project = "python-evdev" -copyright = "2012-2024, Georgi Valkov and contributors" +copyright = "2012-2025, Georgi Valkov and contributors" # The version info for the project you're documenting, acts as replacement for # |version| and |release|, also used in various other places throughout the diff --git a/pyproject.toml b/pyproject.toml index e5e5d00..7854d91 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -29,9 +29,6 @@ classifiers = [ [project.urls] "Homepage" = "https://github.com/gvalkov/python-evdev" -[tool.setuptools] -packages = ["evdev"] - [tool.ruff] line-length = 120 diff --git a/setup.py b/setup.py index e6e0c1d..c5ab4a0 100755 --- a/setup.py +++ b/setup.py @@ -10,7 +10,7 @@ curdir = Path(__file__).resolve().parent -ecodes_c_path = curdir / "evdev/ecodes.c" +ecodes_c_path = curdir / "src/evdev/ecodes.c" def create_ecodes(headers=None): @@ -60,7 +60,7 @@ def create_ecodes(headers=None): print("writing %s (using %s)" % (ecodes_c_path, " ".join(headers))) with ecodes_c_path.open("w") as fh: - cmd = [sys.executable, "evdev/genecodes_c.py", "--ecodes", *headers] + cmd = [sys.executable, "src/evdev/genecodes_c.py", "--ecodes", *headers] run(cmd, check=True, stdout=fh) @@ -93,12 +93,12 @@ def generate_ecodes_py(self): ecodes_py = Path(self.build_lib) / "evdev/ecodes.py" print(f"writing {ecodes_py}") with ecodes_py.open("w") as fh: - cmd = [sys.executable, "-B", "evdev/genecodes_py.py"] + cmd = [sys.executable, "-B", "src/evdev/genecodes_py.py"] res = run(cmd, env={"PYTHONPATH": self.build_lib}, stdout=fh) if res.returncode != 0: print(f"failed to generate static {ecodes_py} - will use ecodes_runtime.py") - shutil.copy("evdev/ecodes_runtime.py", ecodes_py) + shutil.copy("src/evdev/ecodes_runtime.py", ecodes_py) def run(self): for cmd_name in self.get_sub_commands(): @@ -112,9 +112,9 @@ def run(self): cflags = ["-std=c99", "-Wno-error=declaration-after-statement"] setup( ext_modules=[ - Extension("evdev._input", sources=["evdev/input.c"], extra_compile_args=cflags), - Extension("evdev._uinput", sources=["evdev/uinput.c"], extra_compile_args=cflags), - Extension("evdev._ecodes", sources=["evdev/ecodes.c"], extra_compile_args=cflags), + Extension("evdev._input", sources=["src/evdev/input.c"], extra_compile_args=cflags), + Extension("evdev._uinput", sources=["src/evdev/uinput.c"], extra_compile_args=cflags), + Extension("evdev._ecodes", sources=["src/evdev/ecodes.c"], extra_compile_args=cflags), ], cmdclass={ "build_ext": build_ext, diff --git a/evdev/__init__.py b/src/evdev/__init__.py similarity index 100% rename from evdev/__init__.py rename to src/evdev/__init__.py diff --git a/evdev/device.py b/src/evdev/device.py similarity index 100% rename from evdev/device.py rename to src/evdev/device.py diff --git a/evdev/ecodes.py b/src/evdev/ecodes.py similarity index 100% rename from evdev/ecodes.py rename to src/evdev/ecodes.py diff --git a/evdev/ecodes_runtime.py b/src/evdev/ecodes_runtime.py similarity index 100% rename from evdev/ecodes_runtime.py rename to src/evdev/ecodes_runtime.py diff --git a/evdev/eventio.py b/src/evdev/eventio.py similarity index 100% rename from evdev/eventio.py rename to src/evdev/eventio.py diff --git a/evdev/eventio_async.py b/src/evdev/eventio_async.py similarity index 100% rename from evdev/eventio_async.py rename to src/evdev/eventio_async.py diff --git a/evdev/events.py b/src/evdev/events.py similarity index 100% rename from evdev/events.py rename to src/evdev/events.py diff --git a/evdev/evtest.py b/src/evdev/evtest.py similarity index 100% rename from evdev/evtest.py rename to src/evdev/evtest.py diff --git a/evdev/ff.py b/src/evdev/ff.py similarity index 100% rename from evdev/ff.py rename to src/evdev/ff.py diff --git a/evdev/genecodes_c.py b/src/evdev/genecodes_c.py similarity index 100% rename from evdev/genecodes_c.py rename to src/evdev/genecodes_c.py diff --git a/evdev/genecodes_py.py b/src/evdev/genecodes_py.py similarity index 100% rename from evdev/genecodes_py.py rename to src/evdev/genecodes_py.py diff --git a/evdev/input.c b/src/evdev/input.c similarity index 100% rename from evdev/input.c rename to src/evdev/input.c diff --git a/evdev/uinput.c b/src/evdev/uinput.c similarity index 100% rename from evdev/uinput.c rename to src/evdev/uinput.c diff --git a/evdev/uinput.py b/src/evdev/uinput.py similarity index 100% rename from evdev/uinput.py rename to src/evdev/uinput.py diff --git a/evdev/util.py b/src/evdev/util.py similarity index 100% rename from evdev/util.py rename to src/evdev/util.py From 64c6555101b5172d4194c7f92728c1638c613200 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 18:12:47 +0100 Subject: [PATCH 16/40] Optimize reading of events - Construct tuple directly instead of using Py_BuildValue. - Return a tuple of tuples instead of a list of tuples. - Read argument (fd) directly instead of using PyArg_ParseTuple. --- src/evdev/input.c | 43 +++++++++++++++++-------------------------- 1 file changed, 17 insertions(+), 26 deletions(-) diff --git a/src/evdev/input.c b/src/evdev/input.c index 0256745..55e6808 100644 --- a/src/evdev/input.c +++ b/src/evdev/input.c @@ -46,12 +46,10 @@ int test_bit(const char* bitmask, int bit) { static PyObject * device_read(PyObject *self, PyObject *args) { - int fd; struct input_event event; // get device file descriptor (O_RDONLY|O_NONBLOCK) - if (PyArg_ParseTuple(args, "i", &fd) < 0) - return NULL; + int fd = (int)PyLong_AsLong(PyTuple_GET_ITEM(args, 0)); int n = read(fd, &event, sizeof(event)); @@ -68,12 +66,9 @@ device_read(PyObject *self, PyObject *args) PyObject* sec = PyLong_FromLong(event.input_event_sec); PyObject* usec = PyLong_FromLong(event.input_event_usec); PyObject* val = PyLong_FromLong(event.value); - PyObject* py_input_event = NULL; - - py_input_event = Py_BuildValue("OOhhO", sec, usec, event.type, event.code, val); - Py_DECREF(sec); - Py_DECREF(usec); - Py_DECREF(val); + PyObject* type = PyLong_FromLong(event.type); + PyObject* code = PyLong_FromLong(event.code); + PyObject* py_input_event = PyTuple_Pack(5, sec, usec, type, code, val); return py_input_event; } @@ -83,17 +78,16 @@ device_read(PyObject *self, PyObject *args) static PyObject * device_read_many(PyObject *self, PyObject *args) { - int fd; - // get device file descriptor (O_RDONLY|O_NONBLOCK) - int ret = PyArg_ParseTuple(args, "i", &fd); - if (!ret) return NULL; + int fd = (int)PyLong_AsLong(PyTuple_GET_ITEM(args, 0)); - PyObject* event_list = PyList_New(0); PyObject* py_input_event = NULL; + PyObject* events = NULL; PyObject* sec = NULL; PyObject* usec = NULL; PyObject* val = NULL; + PyObject* type = NULL; + PyObject* code = NULL; struct input_event event[64]; @@ -102,26 +96,24 @@ device_read_many(PyObject *self, PyObject *args) if (nread < 0) { PyErr_SetFromErrno(PyExc_OSError); - Py_DECREF(event_list); return NULL; } - // Construct a list of event tuples, which we'll make sense of in Python - for (unsigned i = 0 ; i < nread/event_size ; i++) { + // Construct a tuple of event tuples. Each tuple is the arguments to InputEvent. + size_t num_events = nread / event_size; + events = PyTuple_New(num_events); + for (size_t i = 0 ; i < num_events; i++) { sec = PyLong_FromLong(event[i].input_event_sec); usec = PyLong_FromLong(event[i].input_event_usec); val = PyLong_FromLong(event[i].value); + type = PyLong_FromLong(event[i].type); + code = PyLong_FromLong(event[i].code); - py_input_event = Py_BuildValue("OOhhO", sec, usec, event[i].type, event[i].code, val); - PyList_Append(event_list, py_input_event); - - Py_DECREF(py_input_event); - Py_DECREF(sec); - Py_DECREF(usec); - Py_DECREF(val); + py_input_event = PyTuple_Pack(5, sec, usec, type, code, val); + PyTuple_SET_ITEM(events, i, py_input_event); } - return event_list; + return events; } @@ -539,7 +531,6 @@ ioctl_EVIOCGPROP(PyObject *self, PyObject *args) } - static PyMethodDef MethodTable[] = { { "ioctl_devinfo", ioctl_devinfo, METH_VARARGS, "fetch input device info" }, { "ioctl_capabilities", ioctl_capabilities, METH_VARARGS, "fetch input device capabilities" }, From 0487652d1cb8f32c0f4a3830753b66cc12bab4c5 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 18:25:22 +0100 Subject: [PATCH 17/40] Drop Python 2 support from input.c and uinput.c --- src/evdev/input.c | 24 ++---------------------- src/evdev/uinput.c | 25 ++----------------------- 2 files changed, 4 insertions(+), 45 deletions(-) diff --git a/src/evdev/input.c b/src/evdev/input.c index 55e6808..cfce67c 100644 --- a/src/evdev/input.c +++ b/src/evdev/input.c @@ -552,14 +552,10 @@ static PyMethodDef MethodTable[] = { }; -#define MODULE_NAME "_input" -#define MODULE_HELP "Python bindings to certain linux input subsystem functions" - -#if PY_MAJOR_VERSION >= 3 static struct PyModuleDef moduledef = { PyModuleDef_HEAD_INIT, - MODULE_NAME, - MODULE_HELP, + "_input", + "Python bindings to certain linux input subsystem functions", -1, /* m_size */ MethodTable, /* m_methods */ NULL, /* m_reload */ @@ -581,19 +577,3 @@ PyInit__input(void) { return moduleinit(); } - -#else -static PyObject * -moduleinit(void) -{ - PyObject* m = Py_InitModule3(MODULE_NAME, MethodTable, MODULE_HELP); - if (m == NULL) return NULL; - return m; -} - -PyMODINIT_FUNC -init_input(void) -{ - moduleinit(); -} -#endif diff --git a/src/evdev/uinput.c b/src/evdev/uinput.c index 3494705..8d2c096 100644 --- a/src/evdev/uinput.c +++ b/src/evdev/uinput.c @@ -356,8 +356,6 @@ int _uinput_end_erase(int fd, struct uinput_ff_erase *upload) return ioctl(fd, UI_END_FF_ERASE, upload); } -#define MODULE_NAME "_uinput" -#define MODULE_HELP "Python bindings for parts of linux/uinput.c" static PyMethodDef MethodTable[] = { { "open", uinput_open, METH_VARARGS, @@ -390,11 +388,10 @@ static PyMethodDef MethodTable[] = { { NULL, NULL, 0, NULL} }; -#if PY_MAJOR_VERSION >= 3 static struct PyModuleDef moduledef = { PyModuleDef_HEAD_INIT, - MODULE_NAME, - MODULE_HELP, + "_uinput", + "Python bindings for parts of linux/uinput.c", -1, /* m_size */ MethodTable, /* m_methods */ NULL, /* m_reload */ @@ -418,21 +415,3 @@ PyInit__uinput(void) { return moduleinit(); } - -#else -static PyObject * -moduleinit(void) -{ - PyObject* m = Py_InitModule3(MODULE_NAME, MethodTable, MODULE_HELP); - if (m == NULL) return NULL; - - PyModule_AddIntConstant(m, "maxnamelen", UINPUT_MAX_NAME_SIZE); - return m; -} - -PyMODINIT_FUNC -init_uinput(void) -{ - moduleinit(); -} -#endif From 3e6fd3e24218d868d8c8d1c08b475574b062d08f Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 18:44:40 +0100 Subject: [PATCH 18/40] Update changelog and requirements --- .gitignore | 2 ++ docs/changelog.rst | 10 +++++++++- requirements-dev.txt | 1 + 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index 557f265..70ac303 100644 --- a/.gitignore +++ b/.gitignore @@ -16,6 +16,8 @@ TAGS __pycache__ .pytest_cache .ruff_cache +.venv +uv.lock src/evdev/*.so src/evdev/ecodes.c diff --git a/docs/changelog.rst b/docs/changelog.rst index 92f4e23..8320eae 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -1,9 +1,17 @@ Changelog --------- -1.8.0 (Jan 25, 2025) +1.9.0 (Unreleased) ================== +- Fix ``CPATH/C_INCLUDE_PATH`` being ignored during build. + +- Slightly faster reading of events. + + +1.8.0 (Jan 25, 2025) +==================== + - Binary wheels are now provided by the `evdev-binary `_ package. The package is compiled on manylinux_2_28 against kernel 4.18. diff --git a/requirements-dev.txt b/requirements-dev.txt index 96366e6..725ad7f 100644 --- a/requirements-dev.txt +++ b/requirements-dev.txt @@ -7,3 +7,4 @@ bump-my-version ~= 0.17.4 build twine cibuildwheel +setuptools From 78650f8f50f6a51fe98af9af54d42a121f602e5e Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 20:49:43 +0100 Subject: [PATCH 19/40] FreeBSD related fixes --- docs/changelog.rst | 2 ++ setup.py | 7 ++++++- src/evdev/genecodes_c.py | 3 ++- 3 files changed, 10 insertions(+), 2 deletions(-) diff --git a/docs/changelog.rst b/docs/changelog.rst index 8320eae..20e7293 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -8,6 +8,8 @@ Changelog - Slightly faster reading of events. +- Fix build on FreeBSD. + 1.8.0 (Jan 25, 2025) ==================== diff --git a/setup.py b/setup.py index c5ab4a0..6b721d7 100755 --- a/setup.py +++ b/setup.py @@ -2,6 +2,7 @@ import sys import shutil import textwrap +import platform from pathlib import Path from subprocess import run @@ -25,7 +26,11 @@ def create_ecodes(headers=None): include_paths.update(c_inc_path.split(":")) include_paths.add("/usr/include") - files = ["linux/input.h", "linux/input-event-codes.h", "linux/uinput.h"] + if platform.system().lower() == "freebsd": + files = ["dev/evdev/input.h", "dev/evdev/input-event-codes.h", "dev/evdev/uinput.h"] + else: + files = ["linux/input.h", "linux/input-event-codes.h", "linux/uinput.h"] + headers = [os.path.join(path, file) for path in include_paths for file in files] headers = [header for header in headers if os.path.isfile(header)] diff --git a/src/evdev/genecodes_c.py b/src/evdev/genecodes_c.py index dd3ee91..5c2d946 100644 --- a/src/evdev/genecodes_c.py +++ b/src/evdev/genecodes_c.py @@ -25,7 +25,7 @@ # ----------------------------------------------------------------------------- -macro_regex = r"#define +((?:KEY|ABS|REL|SW|MSC|LED|BTN|REP|SND|ID|EV|BUS|SYN|FF|UI_FF|INPUT_PROP)_\w+)" +macro_regex = r"#define\s+((?:KEY|ABS|REL|SW|MSC|LED|BTN|REP|SND|ID|EV|BUS|SYN|FF|UI_FF|INPUT_PROP)_\w+)" macro_regex = re.compile(macro_regex) # Uname without hostname. @@ -38,6 +38,7 @@ #include #ifdef __FreeBSD__ #include +#include #else #include #include From 5478d94359801adb73d642c412cecd2042677230 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 22:31:20 +0100 Subject: [PATCH 20/40] Give SYN_DROPPED special treatment in evtest and fix alignment --- src/evdev/eventio.py | 2 +- src/evdev/evtest.py | 8 +++++--- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/src/evdev/eventio.py b/src/evdev/eventio.py index 5478f02..5735d18 100644 --- a/src/evdev/eventio.py +++ b/src/evdev/eventio.py @@ -66,7 +66,7 @@ def read(self): `BlockingIOError` if there are no available events at the moment. """ - # events -> [(sec, usec, type, code, val), ...] + # events -> ((sec, usec, type, code, val), ...) events = _input.device_read_many(self.fd) for event in events: diff --git a/src/evdev/evtest.py b/src/evdev/evtest.py index 26e62ad..b0244a9 100644 --- a/src/evdev/evtest.py +++ b/src/evdev/evtest.py @@ -149,9 +149,11 @@ def print_capabilities(device): def print_event(e): if e.type == ecodes.EV_SYN: if e.code == ecodes.SYN_MT_REPORT: - msg = "time {:<16} +++++++++ {} ++++++++" + msg = "time {:<17} +++++++++++++ {} +++++++++++++" + elif e.code == ecodes.SYN_DROPPED: + msg = "time {:<17} !!!!!!!!!!!!! {} !!!!!!!!!!!!!" else: - msg = "time {:<16} --------- {} --------" + msg = "time {:<17} ------------- {} -------------" print(msg.format(e.timestamp(), ecodes.SYN[e.code])) else: if e.type in ecodes.bytype: @@ -159,7 +161,7 @@ def print_event(e): else: codename = "?" - evfmt = "time {:<16} type {} ({}), code {:<4} ({}), value {}" + evfmt = "time {:<17} type {} ({}), code {:<4} ({}), value {}" print(evfmt.format(e.timestamp(), e.type, ecodes.EV[e.type], e.code, codename, e.value)) From 1f083add5e5c377351db976c28fd477507dd7286 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 22:33:17 +0100 Subject: [PATCH 21/40] Drop deprecated InputDevice.fn --- docs/changelog.rst | 2 ++ src/evdev/device.py | 9 --------- 2 files changed, 2 insertions(+), 9 deletions(-) diff --git a/docs/changelog.rst b/docs/changelog.rst index 20e7293..6ad25ed 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -10,6 +10,8 @@ Changelog - Fix build on FreeBSD. +- Drop deprecated ``InputDevice.fn`` (use ``InputDevice.path`` instead). + 1.8.0 (Jan 25, 2025) ==================== diff --git a/src/evdev/device.py b/src/evdev/device.py index 7675a2d..fdd8363 100644 --- a/src/evdev/device.py +++ b/src/evdev/device.py @@ -1,9 +1,6 @@ -# encoding: utf-8 - import collections import contextlib import os -import warnings from . import _input, ecodes, util @@ -383,12 +380,6 @@ def active_keys(self, verbose=False): return active_keys - @property - def fn(self): - msg = "Please use {0}.path instead of {0}.fn".format(self.__class__.__name__) - warnings.warn(msg, DeprecationWarning, stacklevel=2) - return self.path - def absinfo(self, axis_num): """ Return current :class:`AbsInfo` for input device axis From e71716192675fba399cca083e87ac3db64327ca6 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 23:49:30 +0100 Subject: [PATCH 22/40] Use REP_ constants in input.c --- src/evdev/input.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/evdev/input.c b/src/evdev/input.c index cfce67c..4ad0408 100644 --- a/src/evdev/input.c +++ b/src/evdev/input.c @@ -301,7 +301,7 @@ static PyObject * ioctl_EVIOCGREP(PyObject *self, PyObject *args) { int fd, ret; - unsigned int rep[2] = {0}; + unsigned int rep[REP_CNT] = {0}; ret = PyArg_ParseTuple(args, "i", &fd); if (!ret) return NULL; @@ -309,7 +309,7 @@ ioctl_EVIOCGREP(PyObject *self, PyObject *args) if (ret == -1) return NULL; - return Py_BuildValue("(ii)", rep[0], rep[1]); + return Py_BuildValue("(ii)", rep[REP_DELAY], rep[REP_PERIOD]); } @@ -317,7 +317,7 @@ static PyObject * ioctl_EVIOCSREP(PyObject *self, PyObject *args) { int fd, ret; - unsigned int rep[2] = {0}; + unsigned int rep[REP_CNT] = {0}; ret = PyArg_ParseTuple(args, "iii", &fd, &rep[0], &rep[1]); if (!ret) return NULL; From 59dc614a0b3a7046d611aa3ef5f5ee04b4050580 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sun, 2 Feb 2025 23:51:22 +0100 Subject: [PATCH 23/40] More type hints --- docs/changelog.rst | 2 ++ src/evdev/device.py | 74 ++++++++++++++++++++++++-------------------- src/evdev/eventio.py | 9 +++--- src/evdev/uinput.py | 65 +++++++++++++++++++++----------------- src/evdev/util.py | 9 +++--- 5 files changed, 87 insertions(+), 72 deletions(-) diff --git a/docs/changelog.rst b/docs/changelog.rst index 6ad25ed..5dfeaab 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -12,6 +12,8 @@ Changelog - Drop deprecated ``InputDevice.fn`` (use ``InputDevice.path`` instead). +- More type hints. + 1.8.0 (Jan 25, 2025) ==================== diff --git a/src/evdev/device.py b/src/evdev/device.py index fdd8363..73b0acb 100644 --- a/src/evdev/device.py +++ b/src/evdev/device.py @@ -1,6 +1,6 @@ -import collections import contextlib import os +from typing import NamedTuple, Tuple, Union from . import _input, ecodes, util @@ -10,18 +10,10 @@ from .eventio import EvdevError, EventIO -# -------------------------------------------------------------------------- -_AbsInfo = collections.namedtuple("AbsInfo", ["value", "min", "max", "fuzz", "flat", "resolution"]) - -_KbdInfo = collections.namedtuple("KbdInfo", ["delay", "repeat"]) - -_DeviceInfo = collections.namedtuple("DeviceInfo", ["bustype", "vendor", "product", "version"]) - - -class AbsInfo(_AbsInfo): +class AbsInfo(NamedTuple): """Absolute axis information. - A ``namedtuple`` used for storing absolute axis information - + A ``namedtuple`` with absolute axis information - corresponds to the ``input_absinfo`` struct: Attributes @@ -57,11 +49,18 @@ class AbsInfo(_AbsInfo): """ + value: int + min: int + max: int + fuzz: int + flat: int + resolution: int + def __str__(self): - return "val {}, min {}, max {}, fuzz {}, flat {}, res {}".format(*self) + return "value {}, min {}, max {}, fuzz {}, flat {}, res {}".format(*self) # pylint: disable=not-an-iterable -class KbdInfo(_KbdInfo): +class KbdInfo(NamedTuple): """Keyboard repeat rate. Attributes @@ -74,11 +73,14 @@ class KbdInfo(_KbdInfo): Keyboard repeat rate in characters per second. """ + delay: int + repeat: int + def __str__(self): - return "delay {}, repeat {}".format(*self) + return "delay {}, repeat {}".format(self.delay, self.repeat) -class DeviceInfo(_DeviceInfo): +class DeviceInfo(NamedTuple): """ Attributes ---------- @@ -88,9 +90,14 @@ class DeviceInfo(_DeviceInfo): version """ + bustype: int + vendor: int + product: int + version: int + def __str__(self): msg = "bus: {:04x}, vendor {:04x}, product {:04x}, version {:04x}" - return msg.format(*self) + return msg.format(*self) # pylint: disable=not-an-iterable class InputDevice(EventIO): @@ -100,7 +107,7 @@ class InputDevice(EventIO): __slots__ = ("path", "fd", "info", "name", "phys", "uniq", "_rawcapabilities", "version", "ff_effects_count") - def __init__(self, dev): + def __init__(self, dev: Union[str, bytes, os.PathLike]): """ Arguments --------- @@ -111,15 +118,14 @@ def __init__(self, dev): #: Path to input device. self.path = dev if not hasattr(dev, "__fspath__") else dev.__fspath__() - # Certain operations are possible only when the device is opened in - # read-write mode. + # Certain operations are possible only when the device is opened in read-write mode. try: fd = os.open(dev, os.O_RDWR | os.O_NONBLOCK) except OSError: fd = os.open(dev, os.O_RDONLY | os.O_NONBLOCK) #: A non-blocking file descriptor to the device file. - self.fd = fd + self.fd: int = fd # Returns (bustype, vendor, product, version, name, phys, capabilities). info_res = _input.ioctl_devinfo(self.fd) @@ -128,16 +134,16 @@ def __init__(self, dev): self.info = DeviceInfo(*info_res[:4]) #: The name of the event device. - self.name = info_res[4] + self.name: str = info_res[4] #: The physical topology of the device. - self.phys = info_res[5] + self.phys: str = info_res[5] #: The unique identifier of the device. - self.uniq = info_res[6] + self.uniq: str = info_res[6] #: The evdev protocol version. - self.version = _input.ioctl_EVIOCGVERSION(self.fd) + self.version: int = _input.ioctl_EVIOCGVERSION(self.fd) #: The raw dictionary of device capabilities - see `:func:capabilities()`. self._rawcapabilities = _input.ioctl_capabilities(self.fd) @@ -152,7 +158,7 @@ def __del__(self): except (OSError, ImportError, AttributeError): pass - def _capabilities(self, absinfo=True): + def _capabilities(self, absinfo: bool = True): res = {} for etype, _ecodes in self._rawcapabilities.items(): @@ -170,7 +176,7 @@ def _capabilities(self, absinfo=True): return res - def capabilities(self, verbose=False, absinfo=True): + def capabilities(self, verbose: bool = False, absinfo: bool = True): """ Return the event types that this device supports as a mapping of supported event types to lists of handled event codes. @@ -215,7 +221,7 @@ def capabilities(self, verbose=False, absinfo=True): else: return self._capabilities(absinfo) - def input_props(self, verbose=False): + def input_props(self, verbose: bool = False): """ Get device properties and quirks. @@ -236,7 +242,7 @@ def input_props(self, verbose=False): return props - def leds(self, verbose=False): + def leds(self, verbose: bool = False): """ Return currently set LED keys. @@ -257,7 +263,7 @@ def leds(self, verbose=False): return leds - def set_led(self, led_num, value): + def set_led(self, led_num: int, value: int): """ Set the state of the selected LED. @@ -327,7 +333,7 @@ def grab_context(self): yield self.ungrab() - def upload_effect(self, effect): + def upload_effect(self, effect: "ff.Effect"): """ Upload a force feedback effect to a force feedback device. """ @@ -354,10 +360,10 @@ def repeat(self): return KbdInfo(*_input.ioctl_EVIOCGREP(self.fd)) @repeat.setter - def repeat(self, value): + def repeat(self, value: Tuple[int, int]): return _input.ioctl_EVIOCSREP(self.fd, *value) - def active_keys(self, verbose=False): + def active_keys(self, verbose: bool = False): """ Return currently active keys. @@ -380,7 +386,7 @@ def active_keys(self, verbose=False): return active_keys - def absinfo(self, axis_num): + def absinfo(self, axis_num: int): """ Return current :class:`AbsInfo` for input device axis @@ -396,7 +402,7 @@ def absinfo(self, axis_num): """ return AbsInfo(*_input.ioctl_EVIOCGABS(self.fd, axis_num)) - def set_absinfo(self, axis_num, value=None, min=None, max=None, fuzz=None, flat=None, resolution=None): + def set_absinfo(self, axis_num: int, value=None, min=None, max=None, fuzz=None, flat=None, resolution=None): """ Update :class:`AbsInfo` values. Only specified values will be overwritten. diff --git a/src/evdev/eventio.py b/src/evdev/eventio.py index 5735d18..27bba9d 100644 --- a/src/evdev/eventio.py +++ b/src/evdev/eventio.py @@ -2,6 +2,7 @@ import functools import os import select +from typing import Iterator from . import _input, _uinput, ecodes from .events import InputEvent @@ -35,7 +36,7 @@ def fileno(self): """ return self.fd - def read_loop(self): + def read_loop(self) -> Iterator[InputEvent]: """ Enter an endless :func:`select.select()` loop that yields input events. """ @@ -45,7 +46,7 @@ def read_loop(self): for event in self.read(): yield event - def read_one(self): + def read_one(self) -> InputEvent: """ Read and return a single input event as an instance of :class:`InputEvent `. @@ -59,7 +60,7 @@ def read_one(self): if event: return InputEvent(*event) - def read(self): + def read(self) -> Iterator[InputEvent]: """ Read multiple input events from device. Return a generator object that yields :class:`InputEvent ` instances. Raises @@ -114,7 +115,7 @@ def write_event(self, event): self.write(event.type, event.code, event.value) @need_write - def write(self, etype, code, value): + def write(self, etype: int, code: int, value: int): """ Inject an input event into the input subsystem. Events are queued until a synchronization event is received. diff --git a/src/evdev/uinput.py b/src/evdev/uinput.py index 9567374..2c69c2b 100644 --- a/src/evdev/uinput.py +++ b/src/evdev/uinput.py @@ -5,8 +5,10 @@ import stat import time from collections import defaultdict +from typing import Union, Tuple, Dict, Sequence, Optional -from . import _uinput, device, ecodes, ff, util +from . import _uinput, ecodes, ff, util +from .device import InputDevice, AbsInfo from .events import InputEvent try: @@ -38,7 +40,12 @@ class UInput(EventIO): ) @classmethod - def from_device(cls, *devices, filtered_types=(ecodes.EV_SYN, ecodes.EV_FF), **kwargs): + def from_device( + cls, + *devices: Union[InputDevice, Union[str, bytes, os.PathLike]], + filtered_types: Tuple[int] = (ecodes.EV_SYN, ecodes.EV_FF), + **kwargs, + ): """ Create an UInput device with the capabilities of one or more input devices. @@ -57,8 +64,8 @@ def from_device(cls, *devices, filtered_types=(ecodes.EV_SYN, ecodes.EV_FF), **k device_instances = [] for dev in devices: - if not isinstance(dev, device.InputDevice): - dev = device.InputDevice(str(dev)) + if not isinstance(dev, InputDevice): + dev = InputDevice(str(dev)) device_instances.append(dev) all_capabilities = defaultdict(set) @@ -79,14 +86,14 @@ def from_device(cls, *devices, filtered_types=(ecodes.EV_SYN, ecodes.EV_FF), **k def __init__( self, - events=None, - name="py-evdev-uinput", - vendor=0x1, - product=0x1, - version=0x1, - bustype=0x3, - devnode="/dev/uinput", - phys="py-evdev-uinput", + events: Optional[Dict[int, Sequence[int]]] = None, + name: str = "py-evdev-uinput", + vendor: int = 0x1, + product: int = 0x1, + version: int = 0x1, + bustype: int = 0x3, + devnode: str = "/dev/uinput", + phys: str = "py-evdev-uinput", input_props=None, # CentOS 7 has sufficiently old headers that FF_MAX_EFFECTS is not defined there, # which causes the whole module to fail loading. Fallback on a hardcoded value of @@ -131,13 +138,13 @@ def __init__( to inject only ``KEY_*`` and ``BTN_*`` event codes. """ - self.name = name #: Uinput device name. - self.vendor = vendor #: Device vendor identifier. - self.product = product #: Device product identifier. - self.version = version #: Device version identifier. - self.bustype = bustype #: Device bustype - e.g. ``BUS_USB``. - self.phys = phys #: Uinput device physical path. - self.devnode = devnode #: Uinput device node - e.g. ``/dev/uinput/``. + self.name: str = name #: Uinput device name. + self.vendor: int = vendor #: Device vendor identifier. + self.product: int = product #: Device product identifier. + self.version: int = version #: Device version identifier. + self.bustype: int = bustype #: Device bustype - e.g. ``BUS_USB``. + self.phys: str = phys #: Uinput device physical path. + self.devnode: str = devnode #: Uinput device node - e.g. ``/dev/uinput/``. if not events: events = {ecodes.EV_KEY: ecodes.keys.keys()} @@ -173,7 +180,7 @@ def __init__( #: An :class:`InputDevice ` instance #: for the fake input device. ``None`` if the device cannot be #: opened for reading and writing. - self.device = self._find_device(self.fd) + self.device: InputDevice = self._find_device(self.fd) def _prepare_events(self, events): """Prepare events for passing to _uinput.enable and _uinput.setup""" @@ -181,7 +188,7 @@ def _prepare_events(self, events): for etype, codes in events.items(): for code in codes: # Handle max, min, fuzz, flat. - if isinstance(code, (tuple, list, device.AbsInfo)): + if isinstance(code, (tuple, list, AbsInfo)): # Flatten (ABS_Y, (0, 255, 0, 0, 0, 0)) to (ABS_Y, 0, 255, 0, 0, 0, 0). f = [code[0]] f.extend(code[1]) @@ -206,7 +213,7 @@ def __repr__(self): return "{}({})".format(self.__class__.__name__, ", ".join(v)) def __str__(self): - msg = 'name "{}", bus "{}", vendor "{:04x}", product "{:04x}", version "{:04x}", phys "{}"\n' "event types: {}" + msg = 'name "{}", bus "{}", vendor "{:04x}", product "{:04x}", version "{:04x}", phys "{}"\nevent types: {}' evtypes = [i[0] for i in self.capabilities(True).keys()] msg = msg.format( @@ -225,7 +232,7 @@ def close(self): _uinput.close(self.fd) self.fd = -1 - def capabilities(self, verbose=False, absinfo=True): + def capabilities(self, verbose: bool = False, absinfo: bool = True): """See :func:`capabilities `.""" if self.device is None: raise UInputError("input device not opened - cannot read capabilities") @@ -281,7 +288,7 @@ def _verify(self): msg = "uinput device name must not be longer than {} characters" raise UInputError(msg.format(_uinput.maxnamelen)) - def _find_device(self, fd): + def _find_device(self, fd: int) -> InputDevice: """ Tries to find the device node. Will delegate this task to one of several platform-specific functions. @@ -299,7 +306,7 @@ def _find_device(self, fd): # use the generic fallback method. return self._find_device_fallback() - def _find_device_linux(self, sysname): + def _find_device_linux(self, sysname: str) -> InputDevice: """ Tries to find the device node when running on Linux. """ @@ -327,15 +334,15 @@ def _find_device_linux(self, sysname): # device to show up or the permissions to be set. for attempt in range(19): try: - return device.InputDevice(device_path) + return InputDevice(device_path) except (FileNotFoundError, PermissionError): time.sleep(0.1) # Last attempt. If this fails, whatever exception the last attempt raises # shall be the exception that this function raises. - return device.InputDevice(device_path) + return InputDevice(device_path) - def _find_device_fallback(self): + def _find_device_fallback(self) -> Union[InputDevice, None]: """ Tries to find the device node when UI_GET_SYSNAME is not available or we're running on a system sufficiently exotic that we do not know how @@ -363,6 +370,6 @@ def _find_device_fallback(self): path_number_pairs.sort(key=lambda pair: pair[1], reverse=True) for path, _ in path_number_pairs: - d = device.InputDevice(path) + d = InputDevice(path) if d.name == self.name: return d diff --git a/src/evdev/util.py b/src/evdev/util.py index 59991f6..dd7cba6 100644 --- a/src/evdev/util.py +++ b/src/evdev/util.py @@ -3,21 +3,20 @@ import os import re import stat +from typing import Union, List from . import ecodes from .events import event_factory -def list_devices(input_device_dir="/dev/input"): +def list_devices(input_device_dir="/dev/input") -> List[str]: """List readable character devices in ``input_device_dir``.""" fns = glob.glob("{}/event*".format(input_device_dir)) - fns = list(filter(is_device, fns)) + return list(filter(is_device, fns)) - return fns - -def is_device(fn): +def is_device(fn: Union[str, bytes, os.PathLike]) -> bool: """Check if ``fn`` is a readable and writable character device.""" if not os.path.exists(fn): From bc91d17cd6103f37be3d705302132df328737698 Mon Sep 17 00:00:00 2001 From: Sam Bull Date: Tue, 4 Feb 2025 17:50:34 +0000 Subject: [PATCH 24/40] Expose type annotations (#233) * Expose type annotations --- src/evdev/py.typed | 0 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 src/evdev/py.typed diff --git a/src/evdev/py.typed b/src/evdev/py.typed new file mode 100644 index 0000000..e69de29 From 2c623eb5b3b6442ae9cc6a7113d776f23e041cba Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sat, 8 Feb 2025 11:40:49 +0100 Subject: [PATCH 25/40] More type hints --- src/evdev/ecodes.py | 6 +++--- src/evdev/events.py | 37 +++++++++++++++++++------------------ src/evdev/evtest.py | 1 - src/evdev/util.py | 2 +- 4 files changed, 23 insertions(+), 23 deletions(-) diff --git a/src/evdev/ecodes.py b/src/evdev/ecodes.py index a19dcba..fd4afc4 100644 --- a/src/evdev/ecodes.py +++ b/src/evdev/ecodes.py @@ -1,5 +1,5 @@ -# When installed, this module is replaced by an ecodes.py generated at +# When installed, this module is replaced by an ecodes.py generated at # build time by genecodes_py.py (see build_ext in setup.py). -# This stub exists to make development of evdev itself more convenient. -from . ecodes_runtime import * +# This stub exists to make development of evdev itself more convenient. +from .ecodes_runtime import * diff --git a/src/evdev/events.py b/src/evdev/events.py index a4f817d..922bfe6 100644 --- a/src/evdev/events.py +++ b/src/evdev/events.py @@ -38,6 +38,7 @@ # http://www.kernel.org/doc/Documentation/input/event-codes.txt # pylint: disable=no-name-in-module +from typing import Final from .ecodes import ABS, EV_ABS, EV_KEY, EV_REL, EV_SYN, KEY, REL, SYN, keys @@ -48,21 +49,21 @@ class InputEvent: def __init__(self, sec, usec, type, code, value): #: Time in seconds since epoch at which event occurred. - self.sec = sec + self.sec: int = sec #: Microsecond portion of the timestamp. - self.usec = usec + self.usec: int = usec #: Event type - one of ``ecodes.EV_*``. - self.type = type + self.type: int = type #: Event code related to the event type. - self.code = code + self.code: int = code #: Event value related to the event type. - self.value = value + self.value: int = value - def timestamp(self): + def timestamp(self) -> float: """Return event timestamp as a float.""" return self.sec + (self.usec / 1000000.0) @@ -78,20 +79,20 @@ def __repr__(self): class KeyEvent: """An event generated by a keyboard, button or other key-like devices.""" - key_up = 0x0 - key_down = 0x1 - key_hold = 0x2 + key_up: Final[int] = 0x0 + key_down: Final[int] = 0x1 + key_hold: Final[int] = 0x2 __slots__ = "scancode", "keycode", "keystate", "event" - def __init__(self, event, allow_unknown=False): + def __init__(self, event: InputEvent, allow_unknown: bool = False): """ The ``allow_unknown`` argument determines what to do in the event of an event code for which a key code cannot be found. If ``False`` a ``KeyError`` will be raised. If ``True`` the keycode will be set to the hex value of the event code. """ - self.scancode = event.code + self.scancode: int = event.code if event.value == 0: self.keystate = KeyEvent.key_up @@ -109,7 +110,7 @@ def __init__(self, event, allow_unknown=False): raise #: Reference to an :class:`InputEvent` instance. - self.event = event + self.event: InputEvent = event def __str__(self): try: @@ -129,9 +130,9 @@ class RelEvent: __slots__ = "event" - def __init__(self, event): + def __init__(self, event: InputEvent): #: Reference to an :class:`InputEvent` instance. - self.event = event + self.event: InputEvent = event def __str__(self): msg = "relative axis event at {:f}, {}" @@ -146,9 +147,9 @@ class AbsEvent: __slots__ = "event" - def __init__(self, event): + def __init__(self, event: InputEvent): #: Reference to an :class:`InputEvent` instance. - self.event = event + self.event: InputEvent = event def __str__(self): msg = "absolute axis event at {:f}, {}" @@ -166,9 +167,9 @@ class SynEvent: __slots__ = "event" - def __init__(self, event): + def __init__(self, event: InputEvent): #: Reference to an :class:`InputEvent` instance. - self.event = event + self.event: InputEvent = event def __str__(self): msg = "synchronization event at {:f}, {}" diff --git a/src/evdev/evtest.py b/src/evdev/evtest.py index b0244a9..6ea3bb5 100644 --- a/src/evdev/evtest.py +++ b/src/evdev/evtest.py @@ -16,7 +16,6 @@ evtest /dev/input/event0 /dev/input/event1 """ - import atexit import optparse import re diff --git a/src/evdev/util.py b/src/evdev/util.py index dd7cba6..b84ef09 100644 --- a/src/evdev/util.py +++ b/src/evdev/util.py @@ -9,7 +9,7 @@ from .events import event_factory -def list_devices(input_device_dir="/dev/input") -> List[str]: +def list_devices(input_device_dir: Union[str, bytes, os.PathLike] = "/dev/input") -> List[str]: """List readable character devices in ``input_device_dir``.""" fns = glob.glob("{}/event*".format(input_device_dir)) From 7cb02b9c644fbcce69b09375492e39382a1f450c Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sat, 8 Feb 2025 13:36:06 +0100 Subject: [PATCH 26/40] =?UTF-8?q?Bump=20version:=201.8.0=20=E2=86=92=201.9?= =?UTF-8?q?.0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/changelog.rst | 10 +++++----- docs/conf.py | 2 +- pyproject.toml | 4 ++-- src/evdev/eventio.py | 4 ++-- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/docs/changelog.rst b/docs/changelog.rst index 5dfeaab..f66cfff 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -1,18 +1,18 @@ Changelog --------- -1.9.0 (Unreleased) +1.9.0 (Feb 08, 2025) ================== -- Fix ``CPATH/C_INCLUDE_PATH`` being ignored during build. +- Fix for ``CPATH/C_INCLUDE_PATH`` being ignored during build. -- Slightly faster reading of events. +- Slightly faster reading of events in ``device.read()`` and ``device.read_one()``. -- Fix build on FreeBSD. +- Fix FreeBSD support. - Drop deprecated ``InputDevice.fn`` (use ``InputDevice.path`` instead). -- More type hints. +- Improve type hint coverage and add a ``py.typed`` file to the sdist. 1.8.0 (Jan 25, 2025) diff --git a/docs/conf.py b/docs/conf.py index 53b5206..b938fa0 100644 --- a/docs/conf.py +++ b/docs/conf.py @@ -65,7 +65,7 @@ # built documents. # # The full version, including alpha/beta/rc tags. -release = "1.8.0" +release = "1.9.0" # The short X.Y version. version = release diff --git a/pyproject.toml b/pyproject.toml index 7854d91..346dedd 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta" [project] name = "evdev" -version = "1.8.0" +version = "1.9.0" description = "Bindings to the Linux input handling subsystem" keywords = ["evdev", "input", "uinput"] readme = "README.md" @@ -36,7 +36,7 @@ line-length = 120 ignore = ["E265", "E241", "F403", "F401", "E401", "E731"] [tool.bumpversion] -current_version = "1.8.0" +current_version = "1.9.0" commit = true tag = true allow_dirty = true diff --git a/src/evdev/eventio.py b/src/evdev/eventio.py index 27bba9d..bdb91a4 100644 --- a/src/evdev/eventio.py +++ b/src/evdev/eventio.py @@ -2,7 +2,7 @@ import functools import os import select -from typing import Iterator +from typing import Iterator, Union from . import _input, _uinput, ecodes from .events import InputEvent @@ -46,7 +46,7 @@ def read_loop(self) -> Iterator[InputEvent]: for event in self.read(): yield event - def read_one(self) -> InputEvent: + def read_one(self) -> Union[InputEvent, None]: """ Read and return a single input event as an instance of :class:`InputEvent `. From 6523e3f7d77ff5fd15bc2a02033527449b117e72 Mon Sep 17 00:00:00 2001 From: Sam Bull Date: Thu, 20 Feb 2025 07:48:34 +0000 Subject: [PATCH 27/40] Explicit export (#236) --- src/evdev/__init__.py | 27 ++++++++++++++++++++++----- 1 file changed, 22 insertions(+), 5 deletions(-) diff --git a/src/evdev/__init__.py b/src/evdev/__init__.py index 6aa6ef2..5d056f0 100644 --- a/src/evdev/__init__.py +++ b/src/evdev/__init__.py @@ -2,8 +2,25 @@ # Gather everything into a single, convenient namespace. # -------------------------------------------------------------------------- -from . import ecodes, ff -from .device import AbsInfo, DeviceInfo, EvdevError, InputDevice -from .events import AbsEvent, InputEvent, KeyEvent, RelEvent, SynEvent, event_factory -from .uinput import UInput, UInputError -from .util import categorize, list_devices, resolve_ecodes, resolve_ecodes_dict +from . import ecodes as ecodes, ff as ff +from .device import ( + AbsInfo as AbsInfo, + DeviceInfo as DeviceInfo, + EvdevError as EvdevError, + InputDevice as InputDevice, +) +from .events import ( + AbsEvent as AbsEvent, + InputEvent as InputEvent, + KeyEvent as KeyEvent, + RelEvent as RelEvent, + SynEvent as SynEvent, + event_factory as event_factory, +) +from .uinput import UInput as UInput, UInputError as UInputError +from .util import ( + categorize as categorize, + list_devices as list_devices, + resolve_ecodes as resolve_ecodes, + resolve_ecodes_dict as resolve_ecodes_dict, +) From 7916a7beb16f13cb0827e712aa3e889d38ea67e2 Mon Sep 17 00:00:00 2001 From: Sam Bull Date: Thu, 20 Feb 2025 20:56:45 +0000 Subject: [PATCH 28/40] Fill in some type annotations (#237) --- src/evdev/device.py | 32 ++++++++------ src/evdev/eventio_async.py | 87 ++++++++++++++++++++------------------ src/evdev/util.py | 4 +- 3 files changed, 68 insertions(+), 55 deletions(-) diff --git a/src/evdev/device.py b/src/evdev/device.py index 73b0acb..878a937 100644 --- a/src/evdev/device.py +++ b/src/evdev/device.py @@ -1,6 +1,6 @@ import contextlib import os -from typing import NamedTuple, Tuple, Union +from typing import Dict, Iterator, List, Literal, NamedTuple, Tuple, Union, overload from . import _input, ecodes, util @@ -95,7 +95,7 @@ class DeviceInfo(NamedTuple): product: int version: int - def __str__(self): + def __str__(self) -> str: msg = "bus: {:04x}, vendor {:04x}, product {:04x}, version {:04x}" return msg.format(*self) # pylint: disable=not-an-iterable @@ -151,7 +151,7 @@ def __init__(self, dev: Union[str, bytes, os.PathLike]): #: The number of force feedback effects the device can keep in its memory. self.ff_effects_count = _input.ioctl_EVIOCGEFFECTS(self.fd) - def __del__(self): + def __del__(self) -> None: if hasattr(self, "fd") and self.fd is not None: try: self.close() @@ -176,7 +176,13 @@ def _capabilities(self, absinfo: bool = True): return res - def capabilities(self, verbose: bool = False, absinfo: bool = True): + @overload + def capabilities(self, verbose: Literal[False] = ..., absinfo: bool = ...) -> Dict[int, List[int]]: + ... + @overload + def capabilities(self, verbose: Literal[True], absinfo: bool = ...) -> Dict[Tuple[str, int], List[Tuple[str, int]]]: + ... + def capabilities(self, verbose: bool = False, absinfo: bool = True) -> Union[Dict[int, List[int]], Dict[Tuple[str, int], List[Tuple[str, int]]]]: """ Return the event types that this device supports as a mapping of supported event types to lists of handled event codes. @@ -263,7 +269,7 @@ def leds(self, verbose: bool = False): return leds - def set_led(self, led_num: int, value: int): + def set_led(self, led_num: int, value: int) -> None: """ Set the state of the selected LED. @@ -279,18 +285,18 @@ def __eq__(self, other): """ return isinstance(other, self.__class__) and self.info == other.info and self.path == other.path - def __str__(self): + def __str__(self) -> str: msg = 'device {}, name "{}", phys "{}", uniq "{}"' return msg.format(self.path, self.name, self.phys, self.uniq or "") - def __repr__(self): + def __repr__(self) -> str: msg = (self.__class__.__name__, self.path) return "{}({!r})".format(*msg) def __fspath__(self): return self.path - def close(self): + def close(self) -> None: if self.fd > -1: try: super().close() @@ -298,7 +304,7 @@ def close(self): finally: self.fd = -1 - def grab(self): + def grab(self) -> None: """ Grab input device using ``EVIOCGRAB`` - other applications will be unable to receive events until the device is released. Only @@ -311,7 +317,7 @@ def grab(self): _input.ioctl_EVIOCGRAB(self.fd, 1) - def ungrab(self): + def ungrab(self) -> None: """ Release device if it has been already grabbed (uses `EVIOCGRAB`). @@ -324,7 +330,7 @@ def ungrab(self): _input.ioctl_EVIOCGRAB(self.fd, 0) @contextlib.contextmanager - def grab_context(self): + def grab_context(self) -> Iterator[None]: """ A context manager for the duration of which only the current process will be able to receive events from the device. @@ -342,7 +348,7 @@ def upload_effect(self, effect: "ff.Effect"): ff_id = _input.upload_effect(self.fd, data) return ff_id - def erase_effect(self, ff_id): + def erase_effect(self, ff_id) -> None: """ Erase a force effect from a force feedback device. This also stops the effect. @@ -402,7 +408,7 @@ def absinfo(self, axis_num: int): """ return AbsInfo(*_input.ioctl_EVIOCGABS(self.fd, axis_num)) - def set_absinfo(self, axis_num: int, value=None, min=None, max=None, fuzz=None, flat=None, resolution=None): + def set_absinfo(self, axis_num: int, value=None, min=None, max=None, fuzz=None, flat=None, resolution=None) -> None: """ Update :class:`AbsInfo` values. Only specified values will be overwritten. diff --git a/src/evdev/eventio_async.py b/src/evdev/eventio_async.py index fb8bcd2..4af1aab 100644 --- a/src/evdev/eventio_async.py +++ b/src/evdev/eventio_async.py @@ -1,11 +1,57 @@ import asyncio import select +import sys from . import eventio +from .events import InputEvent # needed for compatibility from .eventio import EvdevError +if sys.version_info >= (3, 11): + from typing import Self +else: + from typing import Any as Self + + +class ReadIterator: + def __init__(self, device): + self.current_batch = iter(()) + self.device = device + + # Standard iterator protocol. + def __iter__(self) -> Self: + return self + + def __next__(self) -> InputEvent: + try: + # Read from the previous batch of events. + return next(self.current_batch) + except StopIteration: + r, w, x = select.select([self.device.fd], [], []) + self.current_batch = self.device.read() + return next(self.current_batch) + + def __aiter__(self) -> Self: + return self + + def __anext__(self) -> "asyncio.Future[InputEvent]": + future = asyncio.Future() + try: + # Read from the previous batch of events. + future.set_result(next(self.current_batch)) + except StopIteration: + + def next_batch_ready(batch): + try: + self.current_batch = batch.result() + future.set_result(next(self.current_batch)) + except Exception as e: + future.set_exception(e) + + self.device.async_read().add_done_callback(next_batch_ready) + return future + class EventIO(eventio.EventIO): def _do_when_readable(self, callback): @@ -42,7 +88,7 @@ def async_read(self): self._do_when_readable(lambda: self._set_result(future, self.read)) return future - def async_read_loop(self): + def async_read_loop(self) -> ReadIterator: """ Return an iterator that yields input events. This iterator is compatible with the ``async for`` syntax. @@ -58,42 +104,3 @@ def close(self): # no event loop present, so there is nothing to # remove the reader from. Ignore pass - - -class ReadIterator: - def __init__(self, device): - self.current_batch = iter(()) - self.device = device - - # Standard iterator protocol. - def __iter__(self): - return self - - def __next__(self): - try: - # Read from the previous batch of events. - return next(self.current_batch) - except StopIteration: - r, w, x = select.select([self.device.fd], [], []) - self.current_batch = self.device.read() - return next(self.current_batch) - - def __aiter__(self): - return self - - def __anext__(self): - future = asyncio.Future() - try: - # Read from the previous batch of events. - future.set_result(next(self.current_batch)) - except StopIteration: - - def next_batch_ready(batch): - try: - self.current_batch = batch.result() - future.set_result(next(self.current_batch)) - except Exception as e: - future.set_exception(e) - - self.device.async_read().add_done_callback(next_batch_ready) - return future diff --git a/src/evdev/util.py b/src/evdev/util.py index b84ef09..f873655 100644 --- a/src/evdev/util.py +++ b/src/evdev/util.py @@ -6,7 +6,7 @@ from typing import Union, List from . import ecodes -from .events import event_factory +from .events import InputEvent, event_factory def list_devices(input_device_dir: Union[str, bytes, os.PathLike] = "/dev/input") -> List[str]: @@ -32,7 +32,7 @@ def is_device(fn: Union[str, bytes, os.PathLike]) -> bool: return True -def categorize(event): +def categorize(event: InputEvent) -> InputEvent: """ Categorize an event according to its type. From a98b68f9ac7ac32dbd175f6090e2458ec612f75c Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Fri, 21 Feb 2025 18:24:49 +0100 Subject: [PATCH 29/40] Fix for UI_FF constants missing from generated ecodes.py --- .github/workflows/test.yml | 1 - src/evdev/__init__.py | 17 +++++++++++++++-- src/evdev/ecodes_runtime.py | 2 +- src/evdev/genecodes_py.py | 3 ++- tests/test_ecodes.py | 16 +++++++++++++--- 5 files changed, 31 insertions(+), 8 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 3ee56d3..b9cd26d 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -21,7 +21,6 @@ jobs: python-version: ${{ matrix.python-version }} - name: Run pytest tests - # pip install -e . builds _ecodes and such into the evdev directory # sudo required to write to uinputs run: | sudo python -m pip install pytest setuptools diff --git a/src/evdev/__init__.py b/src/evdev/__init__.py index 5d056f0..bae0fec 100644 --- a/src/evdev/__init__.py +++ b/src/evdev/__init__.py @@ -2,13 +2,21 @@ # Gather everything into a single, convenient namespace. # -------------------------------------------------------------------------- -from . import ecodes as ecodes, ff as ff +# The superfluous "import name as name" syntax is here to satisfy mypy's attrs-defined rule. +# Alternatively all exported objects can be listed in __all__. + +from . import ( + ecodes as ecodes, + ff as ff, +) + from .device import ( AbsInfo as AbsInfo, DeviceInfo as DeviceInfo, EvdevError as EvdevError, InputDevice as InputDevice, ) + from .events import ( AbsEvent as AbsEvent, InputEvent as InputEvent, @@ -17,7 +25,12 @@ SynEvent as SynEvent, event_factory as event_factory, ) -from .uinput import UInput as UInput, UInputError as UInputError + +from .uinput import ( + UInput as UInput, + UInputError as UInputError, +) + from .util import ( categorize as categorize, list_devices as list_devices, diff --git a/src/evdev/ecodes_runtime.py b/src/evdev/ecodes_runtime.py index d6c8b2a..47f3b23 100644 --- a/src/evdev/ecodes_runtime.py +++ b/src/evdev/ecodes_runtime.py @@ -46,7 +46,7 @@ #: Mapping of names to values. ecodes = {} -prefixes = "KEY ABS REL SW MSC LED BTN REP SND ID EV BUS SYN FF_STATUS FF INPUT_PROP".split() +prefixes = "KEY ABS REL SW MSC LED BTN REP SND ID EV BUS SYN FF_STATUS FF INPUT_PROP UI_FF".split() prev_prefix = "" g = globals() diff --git a/src/evdev/genecodes_py.py b/src/evdev/genecodes_py.py index 1afbc34..f00020c 100644 --- a/src/evdev/genecodes_py.py +++ b/src/evdev/genecodes_py.py @@ -40,6 +40,7 @@ ("BUS", "Dict[int, Union[str, Tuple[str]]]", None), ("SYN", "Dict[int, Union[str, Tuple[str]]]", None), ("FF", "Dict[int, Union[str, Tuple[str]]]", None), + ("UI_FF", "Dict[int, Union[str, Tuple[str]]]", None), ("FF_STATUS", "Dict[int, Union[str, Tuple[str]]]", None), ("INPUT_PROP", "Dict[int, Union[str, Tuple[str]]]", None) ] @@ -50,4 +51,4 @@ print(f"{key}: {annotation} = ", end="") pprint(getattr(ecodes, key)) - print() + print() \ No newline at end of file diff --git a/tests/test_ecodes.py b/tests/test_ecodes.py index c810b4f..5c3e38d 100644 --- a/tests/test_ecodes.py +++ b/tests/test_ecodes.py @@ -1,9 +1,8 @@ -# encoding: utf-8 - from evdev import ecodes +from evdev import ecodes_runtime -prefixes = "KEY ABS REL SW MSC LED BTN REP SND ID EV BUS SYN FF_STATUS FF" +prefixes = "KEY ABS REL SW MSC LED BTN REP SND ID EV BUS SYN FF_STATUS FF UI_FF" def to_tuples(val): @@ -29,3 +28,14 @@ def test_overlap(): vals_ff = set(to_tuples(ecodes.FF.values())) vals_ff_status = set(to_tuples(ecodes.FF_STATUS.values())) assert bool(vals_ff & vals_ff_status) is False + + +def test_generated(): + e_run = vars(ecodes_runtime) + e_gen = vars(ecodes) + + def keys(v): + res = {k for k in v.keys() if not k.startswith("_") and not k[1].islower()} + return res + + assert keys(e_run) == keys(e_gen) \ No newline at end of file From 82d09f631a16329c6d3ca2a2fd3789fac3a01c4b Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Sat, 22 Feb 2025 12:05:08 +0100 Subject: [PATCH 30/40] =?UTF-8?q?Bump=20version:=201.9.0=20=E2=86=92=201.9?= =?UTF-8?q?.1?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/changelog.rst | 11 ++++++++++- docs/conf.py | 2 +- pyproject.toml | 4 ++-- 3 files changed, 13 insertions(+), 4 deletions(-) diff --git a/docs/changelog.rst b/docs/changelog.rst index f66cfff..4dcf62f 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -1,8 +1,17 @@ Changelog --------- + +1.9.1 (Feb 22, 2025) +==================== + +- Fix fox missing ``UI_FF`` constants in generated ``ecodes.py``. + +- More type annotations. + + 1.9.0 (Feb 08, 2025) -================== +==================== - Fix for ``CPATH/C_INCLUDE_PATH`` being ignored during build. diff --git a/docs/conf.py b/docs/conf.py index b938fa0..86b3d06 100644 --- a/docs/conf.py +++ b/docs/conf.py @@ -65,7 +65,7 @@ # built documents. # # The full version, including alpha/beta/rc tags. -release = "1.9.0" +release = "1.9.1" # The short X.Y version. version = release diff --git a/pyproject.toml b/pyproject.toml index 346dedd..d248ab2 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta" [project] name = "evdev" -version = "1.9.0" +version = "1.9.1" description = "Bindings to the Linux input handling subsystem" keywords = ["evdev", "input", "uinput"] readme = "README.md" @@ -36,7 +36,7 @@ line-length = 120 ignore = ["E265", "E241", "F403", "F401", "E401", "E731"] [tool.bumpversion] -current_version = "1.9.0" +current_version = "1.9.1" commit = true tag = true allow_dirty = true From 5f9fd2cd11daa9a54452dadbf00aaf284d3d6063 Mon Sep 17 00:00:00 2001 From: bastian-wattro <106541220+bastian-wattro@users.noreply.github.com> Date: Fri, 28 Feb 2025 08:41:31 +0100 Subject: [PATCH 31/40] fix utils.categorize return type (#240) --- src/evdev/util.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/evdev/util.py b/src/evdev/util.py index f873655..db89a22 100644 --- a/src/evdev/util.py +++ b/src/evdev/util.py @@ -6,7 +6,7 @@ from typing import Union, List from . import ecodes -from .events import InputEvent, event_factory +from .events import InputEvent, event_factory, KeyEvent, RelEvent, AbsEvent, SynEvent def list_devices(input_device_dir: Union[str, bytes, os.PathLike] = "/dev/input") -> List[str]: @@ -32,7 +32,7 @@ def is_device(fn: Union[str, bytes, os.PathLike]) -> bool: return True -def categorize(event: InputEvent) -> InputEvent: +def categorize(event: InputEvent) -> Union[InputEvent, KeyEvent, RelEvent, AbsEvent, SynEvent]: """ Categorize an event according to its type. From 6b4e8ef0ee505d9c3d46b1787eac339d8bd0b934 Mon Sep 17 00:00:00 2001 From: Yoann Congal Date: Thu, 1 May 2025 19:42:17 +0200 Subject: [PATCH 32/40] Add a reproducibility option for building ecodes.c (#242) ecodes.c currently contains the kernel info of the build machine and the full path of the input*.h headers: This is not reproducible as output can change even is headers content do not. Downstream distributions might package ecodes.c and get non-reproducible output. To fix this: introduce a --reproducible option in the build: - in setup.py build_ecodes command - in underlying genecodes_c.py Note: These options are disabled by default so no change is expected in current builds. Signed-off-by: Yoann Congal --- setup.py | 13 ++++++++++--- src/evdev/genecodes_c.py | 17 +++++++++++------ 2 files changed, 21 insertions(+), 9 deletions(-) diff --git a/setup.py b/setup.py index 6b721d7..3371199 100755 --- a/setup.py +++ b/setup.py @@ -14,7 +14,7 @@ ecodes_c_path = curdir / "src/evdev/ecodes.c" -def create_ecodes(headers=None): +def create_ecodes(headers=None, reproducibility=False): if not headers: include_paths = set() cpath = os.environ.get("CPATH", "").strip() @@ -65,7 +65,10 @@ def create_ecodes(headers=None): print("writing %s (using %s)" % (ecodes_c_path, " ".join(headers))) with ecodes_c_path.open("w") as fh: - cmd = [sys.executable, "src/evdev/genecodes_c.py", "--ecodes", *headers] + cmd = [sys.executable, "src/evdev/genecodes_c.py"] + if reproducibility: + cmd.append("--reproducibility") + cmd.extend(["--ecodes", *headers]) run(cmd, check=True, stdout=fh) @@ -74,17 +77,21 @@ class build_ecodes(Command): user_options = [ ("evdev-headers=", None, "colon-separated paths to input subsystem headers"), + ("reproducibility", None, "hide host details (host/paths) to create a reproducible output"), ] def initialize_options(self): self.evdev_headers = None + self.reproducibility = False def finalize_options(self): if self.evdev_headers: self.evdev_headers = self.evdev_headers.split(":") + if self.reproducibility is None: + self.reproducibility = False def run(self): - create_ecodes(self.evdev_headers) + create_ecodes(self.evdev_headers, reproducibility=self.reproducibility) class build_ext(_build_ext.build_ext): diff --git a/src/evdev/genecodes_c.py b/src/evdev/genecodes_c.py index 5c2d946..24cad27 100644 --- a/src/evdev/genecodes_c.py +++ b/src/evdev/genecodes_c.py @@ -15,22 +15,27 @@ "/usr/include/linux/uinput.h", ] -opts, args = getopt.getopt(sys.argv[1:], "", ["ecodes", "stubs"]) +opts, args = getopt.getopt(sys.argv[1:], "", ["ecodes", "stubs", "reproducibility"]) if not opts: - print("usage: genecodes.py [--ecodes|--stubs] ") + print("usage: genecodes.py [--ecodes|--stubs] [--reproducibility] ") exit(2) if args: headers = args +reproducibility = ("--reproducibility", "") in opts + # ----------------------------------------------------------------------------- macro_regex = r"#define\s+((?:KEY|ABS|REL|SW|MSC|LED|BTN|REP|SND|ID|EV|BUS|SYN|FF|UI_FF|INPUT_PROP)_\w+)" macro_regex = re.compile(macro_regex) -# Uname without hostname. -uname = list(os.uname()) -uname = " ".join((uname[0], *uname[2:])) +if reproducibility: + uname = "hidden for reproducibility" +else: + # Uname without hostname. + uname = list(os.uname()) + uname = " ".join((uname[0], *uname[2:])) # ----------------------------------------------------------------------------- @@ -138,5 +143,5 @@ def parse_headers(headers=headers): template = template_stubs body = os.linesep.join(body) -text = template % (uname, headers, body) +text = template % (uname, headers if not reproducibility else ["hidden for reproducibility"], body) print(text.strip()) From 3bc969bf59e842c9e2f9569f39434f77b911224f Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Thu, 1 May 2025 22:14:44 +0300 Subject: [PATCH 33/40] s/reproducibility/reproducible --- setup.py | 16 ++++++++-------- src/evdev/genecodes_c.py | 10 +++++----- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/setup.py b/setup.py index 3371199..1f6eaac 100755 --- a/setup.py +++ b/setup.py @@ -14,7 +14,7 @@ ecodes_c_path = curdir / "src/evdev/ecodes.c" -def create_ecodes(headers=None, reproducibility=False): +def create_ecodes(headers=None, reproducible=False): if not headers: include_paths = set() cpath = os.environ.get("CPATH", "").strip() @@ -66,8 +66,8 @@ def create_ecodes(headers=None, reproducibility=False): print("writing %s (using %s)" % (ecodes_c_path, " ".join(headers))) with ecodes_c_path.open("w") as fh: cmd = [sys.executable, "src/evdev/genecodes_c.py"] - if reproducibility: - cmd.append("--reproducibility") + if reproducible: + cmd.append("--reproducible") cmd.extend(["--ecodes", *headers]) run(cmd, check=True, stdout=fh) @@ -77,21 +77,21 @@ class build_ecodes(Command): user_options = [ ("evdev-headers=", None, "colon-separated paths to input subsystem headers"), - ("reproducibility", None, "hide host details (host/paths) to create a reproducible output"), + ("reproducible", None, "hide host details (host/paths) to create a reproducible output"), ] def initialize_options(self): self.evdev_headers = None - self.reproducibility = False + self.reproducible = False def finalize_options(self): if self.evdev_headers: self.evdev_headers = self.evdev_headers.split(":") - if self.reproducibility is None: - self.reproducibility = False + if self.reproducible is None: + self.reproducible = False def run(self): - create_ecodes(self.evdev_headers, reproducibility=self.reproducibility) + create_ecodes(self.evdev_headers, reproducible=self.reproducible) class build_ext(_build_ext.build_ext): diff --git a/src/evdev/genecodes_c.py b/src/evdev/genecodes_c.py index 24cad27..15a6693 100644 --- a/src/evdev/genecodes_c.py +++ b/src/evdev/genecodes_c.py @@ -15,22 +15,22 @@ "/usr/include/linux/uinput.h", ] -opts, args = getopt.getopt(sys.argv[1:], "", ["ecodes", "stubs", "reproducibility"]) +opts, args = getopt.getopt(sys.argv[1:], "", ["ecodes", "stubs", "reproducible"]) if not opts: - print("usage: genecodes.py [--ecodes|--stubs] [--reproducibility] ") + print("usage: genecodes.py [--ecodes|--stubs] [--reproducible] ") exit(2) if args: headers = args -reproducibility = ("--reproducibility", "") in opts +reproducible = ("--reproducible", "") in opts # ----------------------------------------------------------------------------- macro_regex = r"#define\s+((?:KEY|ABS|REL|SW|MSC|LED|BTN|REP|SND|ID|EV|BUS|SYN|FF|UI_FF|INPUT_PROP)_\w+)" macro_regex = re.compile(macro_regex) -if reproducibility: +if reproducible: uname = "hidden for reproducibility" else: # Uname without hostname. @@ -143,5 +143,5 @@ def parse_headers(headers=headers): template = template_stubs body = os.linesep.join(body) -text = template % (uname, headers if not reproducibility else ["hidden for reproducibility"], body) +text = template % (uname, headers if not reproducible else ["hidden for reproducibility"], body) print(text.strip()) From 8f45223a11d0b48b8485e059d63896e65657dea8 Mon Sep 17 00:00:00 2001 From: Sam Bull Date: Thu, 1 May 2025 19:31:49 +0100 Subject: [PATCH 34/40] Use Generic to set precise type for InputDevice.path (#241) * Use Generic to set precise type for InputDevice.path * Update src/evdev/device.py --- src/evdev/device.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/evdev/device.py b/src/evdev/device.py index 878a937..a7f9b92 100644 --- a/src/evdev/device.py +++ b/src/evdev/device.py @@ -1,6 +1,6 @@ import contextlib import os -from typing import Dict, Iterator, List, Literal, NamedTuple, Tuple, Union, overload +from typing import Dict, Generic, Iterator, List, Literal, NamedTuple, Tuple, TypeVar, Union, overload from . import _input, ecodes, util @@ -9,6 +9,8 @@ except ImportError: from .eventio import EvdevError, EventIO +_AnyStr = TypeVar("_AnyStr", str, bytes) + class AbsInfo(NamedTuple): """Absolute axis information. @@ -100,14 +102,14 @@ def __str__(self) -> str: return msg.format(*self) # pylint: disable=not-an-iterable -class InputDevice(EventIO): +class InputDevice(EventIO, Generic[_AnyStr]): """ A linux input device from which input events can be read. """ __slots__ = ("path", "fd", "info", "name", "phys", "uniq", "_rawcapabilities", "version", "ff_effects_count") - def __init__(self, dev: Union[str, bytes, os.PathLike]): + def __init__(self, dev: Union[_AnyStr, "os.PathLike[_AnyStr]"]): """ Arguments --------- @@ -116,7 +118,7 @@ def __init__(self, dev: Union[str, bytes, os.PathLike]): """ #: Path to input device. - self.path = dev if not hasattr(dev, "__fspath__") else dev.__fspath__() + self.path: _AnyStr = dev if not hasattr(dev, "__fspath__") else dev.__fspath__() # Certain operations are possible only when the device is opened in read-write mode. try: From a5d8cf0749f15d44feb76bbed27b30a75b3c7c1f Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Thu, 1 May 2025 22:15:19 +0300 Subject: [PATCH 35/40] =?UTF-8?q?Bump=20version:=201.9.1=20=E2=86=92=201.9?= =?UTF-8?q?.2?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/changelog.rst | 11 +++++++++++ docs/conf.py | 2 +- pyproject.toml | 4 ++-- 3 files changed, 14 insertions(+), 3 deletions(-) diff --git a/docs/changelog.rst b/docs/changelog.rst index 4dcf62f..49f5911 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -2,6 +2,17 @@ Changelog --------- +1.9.2 (May 01, 2025) +==================== + +- Add the "--reproducible" build option which removes the build date and used headers from the + generated ``ecodes.c``. Example usage:: + + python -m build --config-setting=--build-option='build_ecodes --reproducible' -n + +- Use ``Generic`` to set precise type for ``InputDevice.path``. + + 1.9.1 (Feb 22, 2025) ==================== diff --git a/docs/conf.py b/docs/conf.py index 86b3d06..758f878 100644 --- a/docs/conf.py +++ b/docs/conf.py @@ -65,7 +65,7 @@ # built documents. # # The full version, including alpha/beta/rc tags. -release = "1.9.1" +release = "1.9.2" # The short X.Y version. version = release diff --git a/pyproject.toml b/pyproject.toml index d248ab2..e6a6ac7 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta" [project] name = "evdev" -version = "1.9.1" +version = "1.9.2" description = "Bindings to the Linux input handling subsystem" keywords = ["evdev", "input", "uinput"] readme = "README.md" @@ -36,7 +36,7 @@ line-length = 120 ignore = ["E265", "E241", "F403", "F401", "E401", "E731"] [tool.bumpversion] -current_version = "1.9.1" +current_version = "1.9.2" commit = true tag = true allow_dirty = true From 5227b1672cbf074287088860c855c24bb96fe6b1 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Thu, 5 Feb 2026 00:09:01 +0100 Subject: [PATCH 36/40] Fix memory leaks --- src/evdev/input.c | 49 ++++++++++++++++++++++++----------------------- 1 file changed, 25 insertions(+), 24 deletions(-) diff --git a/src/evdev/input.c b/src/evdev/input.c index 4ad0408..894db22 100644 --- a/src/evdev/input.c +++ b/src/evdev/input.c @@ -63,12 +63,12 @@ device_read(PyObject *self, PyObject *args) return NULL; } - PyObject* sec = PyLong_FromLong(event.input_event_sec); - PyObject* usec = PyLong_FromLong(event.input_event_usec); - PyObject* val = PyLong_FromLong(event.value); - PyObject* type = PyLong_FromLong(event.type); - PyObject* code = PyLong_FromLong(event.code); - PyObject* py_input_event = PyTuple_Pack(5, sec, usec, type, code, val); + PyObject *py_input_event = PyTuple_New(5); + PyTuple_SET_ITEM(py_input_event, 0, PyLong_FromLong(event.input_event_sec)); + PyTuple_SET_ITEM(py_input_event, 1, PyLong_FromLong(event.input_event_usec)); + PyTuple_SET_ITEM(py_input_event, 2, PyLong_FromLong(event.type)); + PyTuple_SET_ITEM(py_input_event, 3, PyLong_FromLong(event.code)); + PyTuple_SET_ITEM(py_input_event, 4, PyLong_FromLong(event.value)); return py_input_event; } @@ -81,14 +81,6 @@ device_read_many(PyObject *self, PyObject *args) // get device file descriptor (O_RDONLY|O_NONBLOCK) int fd = (int)PyLong_AsLong(PyTuple_GET_ITEM(args, 0)); - PyObject* py_input_event = NULL; - PyObject* events = NULL; - PyObject* sec = NULL; - PyObject* usec = NULL; - PyObject* val = NULL; - PyObject* type = NULL; - PyObject* code = NULL; - struct input_event event[64]; size_t event_size = sizeof(struct input_event); @@ -101,15 +93,15 @@ device_read_many(PyObject *self, PyObject *args) // Construct a tuple of event tuples. Each tuple is the arguments to InputEvent. size_t num_events = nread / event_size; - events = PyTuple_New(num_events); - for (size_t i = 0 ; i < num_events; i++) { - sec = PyLong_FromLong(event[i].input_event_sec); - usec = PyLong_FromLong(event[i].input_event_usec); - val = PyLong_FromLong(event[i].value); - type = PyLong_FromLong(event[i].type); - code = PyLong_FromLong(event[i].code); - py_input_event = PyTuple_Pack(5, sec, usec, type, code, val); + PyObject* events = PyTuple_New(num_events); + for (size_t i = 0 ; i < num_events; i++) { + PyObject *py_input_event = PyTuple_New(5); + PyTuple_SET_ITEM(py_input_event, 0, PyLong_FromLong(event[i].input_event_sec)); + PyTuple_SET_ITEM(py_input_event, 1, PyLong_FromLong(event[i].input_event_usec)); + PyTuple_SET_ITEM(py_input_event, 2, PyLong_FromLong(event[i].type)); + PyTuple_SET_ITEM(py_input_event, 3, PyLong_FromLong(event[i].code)); + PyTuple_SET_ITEM(py_input_event, 4, PyLong_FromLong(event[i].value)); PyTuple_SET_ITEM(events, i, py_input_event); } @@ -200,6 +192,11 @@ ioctl_capabilities(PyObject *self, PyObject *args) return capabilities; on_err: + Py_XDECREF(capabilities); + Py_XDECREF(eventcodes); + Py_XDECREF(capability); + Py_XDECREF(py_absinfo); + Py_XDECREF(absitem); PyErr_SetFromErrno(PyExc_OSError); return NULL; } @@ -408,7 +405,9 @@ ioctl_EVIOCG_bits(PyObject *self, PyObject *args) PyObject* res = PyList_New(0); for (int i=0; i<=max; i++) { if (test_bit(bytes, i)) { - PyList_Append(res, Py_BuildValue("i", i)); + PyObject *val = PyLong_FromLong(i); + PyList_Append(res, val); + Py_DECREF(val); } } @@ -523,7 +522,9 @@ ioctl_EVIOCGPROP(PyObject *self, PyObject *args) PyObject* res = PyList_New(0); for (int i=0; i Date: Thu, 5 Feb 2026 22:16:48 +0100 Subject: [PATCH 37/40] CI fixes --- .github/workflows/install.yaml | 6 +++--- .github/workflows/lint.yml | 6 +++--- .github/workflows/test.yml | 6 +++--- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/workflows/install.yaml b/.github/workflows/install.yaml index 87502ad..f07c035 100644 --- a/.github/workflows/install.yaml +++ b/.github/workflows/install.yaml @@ -11,15 +11,15 @@ jobs: fail-fast: false matrix: os: [ubuntu-latest] - python-version: ["3.8", "3.9", "3.10", "3.11", "3.12", "3.13"] + python-version: ["3.8", "3.9", "3.10", "3.11", "3.12", "3.13", "3.14"] include: - os: ubuntu-latest python-version: "3.8" steps: - - uses: actions/checkout@v4 + - uses: actions/checkout@v6 - name: Set up Python ${{ matrix.python-version }} - uses: actions/setup-python@v5 + uses: actions/setup-python@v6 with: python-version: ${{ matrix.python-version }} diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index e293976..20d254b 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -11,12 +11,12 @@ jobs: fail-fast: false matrix: os: [ubuntu-latest] - python-version: ["3.12"] + python-version: ["3.14"] steps: - - uses: actions/checkout@v4 + - uses: actions/checkout@v6 - name: Set up Python ${{ matrix.python-version }} - uses: actions/setup-python@v5 + uses: actions/setup-python@v6 with: python-version: ${{ matrix.python-version }} diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index b9cd26d..073d524 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -11,12 +11,12 @@ jobs: fail-fast: false matrix: os: [ubuntu-latest] - python-version: ["3.12"] + python-version: ["3.14"] steps: - - uses: actions/checkout@v4 + - uses: actions/checkout@v6 - name: Set up Python ${{ matrix.python-version }} - uses: actions/setup-python@v5 + uses: actions/setup-python@v6 with: python-version: ${{ matrix.python-version }} From fae2cf9d1d4a3e2700e148399f312b44b6208a56 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Thu, 5 Feb 2026 22:24:12 +0100 Subject: [PATCH 38/40] Use an SPDX license --- pyproject.toml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pyproject.toml b/pyproject.toml index e6a6ac7..665a9b7 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -8,7 +8,7 @@ version = "1.9.2" description = "Bindings to the Linux input handling subsystem" keywords = ["evdev", "input", "uinput"] readme = "README.md" -license = {file = "LICENSE"} +license = "BSD-3-Clause" requires-python = ">=3.8" authors = [ { name="Georgi Valkov", email="georgi.t.valkov@gmail.com" }, @@ -22,7 +22,6 @@ classifiers = [ "Operating System :: POSIX :: Linux", "Intended Audience :: Developers", "Topic :: Software Development :: Libraries", - "License :: OSI Approved :: BSD License", "Programming Language :: Python :: Implementation :: CPython", ] From faf7bc93c6a97edc317c4cc6a8d81ab94e5ba77f Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Thu, 5 Feb 2026 22:36:13 +0100 Subject: [PATCH 39/40] Drop support for Python 3.8 and raise setuptools version to 77.0 --- .github/workflows/install.yaml | 4 ++-- docs/changelog.rst | 7 +++++++ pyproject.toml | 4 ++-- 3 files changed, 11 insertions(+), 4 deletions(-) diff --git a/.github/workflows/install.yaml b/.github/workflows/install.yaml index f07c035..e879179 100644 --- a/.github/workflows/install.yaml +++ b/.github/workflows/install.yaml @@ -11,10 +11,10 @@ jobs: fail-fast: false matrix: os: [ubuntu-latest] - python-version: ["3.8", "3.9", "3.10", "3.11", "3.12", "3.13", "3.14"] + python-version: ["3.9", "3.10", "3.11", "3.12", "3.13", "3.14"] include: - os: ubuntu-latest - python-version: "3.8" + python-version: "3.9" steps: - uses: actions/checkout@v6 diff --git a/docs/changelog.rst b/docs/changelog.rst index 49f5911..bcf1636 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -1,6 +1,13 @@ Changelog --------- +1.9.3 (Feb 05, 2025) +==================== + +- Fix several memory leaks in ``input.c``. + +- Raise the minimum supported Python version to 3.9 and the setuptools version to 77.0. + 1.9.2 (May 01, 2025) ==================== diff --git a/pyproject.toml b/pyproject.toml index 665a9b7..159460c 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -1,5 +1,5 @@ [build-system] -requires = ["setuptools>=61.0"] +requires = ["setuptools>=77.0"] build-backend = "setuptools.build_meta" [project] @@ -9,7 +9,7 @@ description = "Bindings to the Linux input handling subsystem" keywords = ["evdev", "input", "uinput"] readme = "README.md" license = "BSD-3-Clause" -requires-python = ">=3.8" +requires-python = ">=3.9" authors = [ { name="Georgi Valkov", email="georgi.t.valkov@gmail.com" }, ] From a47b5b5a6f79bde6823095d1105501856338aed7 Mon Sep 17 00:00:00 2001 From: Georgi Valkov Date: Thu, 5 Feb 2026 22:46:48 +0100 Subject: [PATCH 40/40] =?UTF-8?q?Bump=20version:=201.9.2=20=E2=86=92=201.9?= =?UTF-8?q?.3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/conf.py | 2 +- pyproject.toml | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/conf.py b/docs/conf.py index 758f878..0be06b3 100644 --- a/docs/conf.py +++ b/docs/conf.py @@ -65,7 +65,7 @@ # built documents. # # The full version, including alpha/beta/rc tags. -release = "1.9.2" +release = "1.9.3" # The short X.Y version. version = release diff --git a/pyproject.toml b/pyproject.toml index 159460c..d0b4f7c 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta" [project] name = "evdev" -version = "1.9.2" +version = "1.9.3" description = "Bindings to the Linux input handling subsystem" keywords = ["evdev", "input", "uinput"] readme = "README.md" @@ -35,7 +35,7 @@ line-length = 120 ignore = ["E265", "E241", "F403", "F401", "E401", "E731"] [tool.bumpversion] -current_version = "1.9.2" +current_version = "1.9.3" commit = true tag = true allow_dirty = true