systemd / pystemd

A thin Cython-based wrapper on top of libsystemd, focused on exposing the dbus API via sd-bus in an automated and easy to consume way.
GNU Lesser General Public License v2.1
409 stars 36 forks source link

Fix deprecation warning: LICENSE is autodetected #87

Closed michel-slm closed 1 year ago

michel-slm commented 1 year ago

license_file can be renamed to license_files, but on the other hand it's not needed anymore as LICENSE gets autodetected:

https://setuptools.pypa.io/en/latest/references/keywords.html#keywords

michel in pystemd on  fix-setup_cfg-deprecated-field via 🐍 v3.11.3 took 5s
⬢ [fedora-toolbox:38] ❯ python3 setup.py --license
Compiling pystemd/cutils.pyx because it changed.
Compiling pystemd/daemon.pyx because it changed.
Compiling pystemd/dbusexc.pyx because it changed.
Compiling pystemd/dbuslib.pyx because it changed.
Compiling pystemd/journal.pyx because it changed.
[1/5] Cythonizing pystemd/cutils.pyx
/usr/lib64/python3.11/site-packages/Cython/Compiler/Main.py:369: FutureWarning: Cython directive 'language_level' not set, using 2 for now (Py2). This will change in a later release! File: /home/michel/src/github/systemd/pystemd/pystemd/cutils.pyx
  tree = Parsing.p_module(s, pxd, full_module_name)
[2/5] Cythonizing pystemd/daemon.pyx
[3/5] Cythonizing pystemd/dbusexc.pyx
[4/5] Cythonizing pystemd/dbuslib.pyx
[5/5] Cythonizing pystemd/journal.pyx
/usr/lib/python3.11/site-packages/setuptools/config/setupcfg.py:508: SetuptoolsDeprecationWarning: The license_file parameter is deprecated, use license_files instead.
  warnings.warn(msg, warning_class)
/usr/lib/python3.11/site-packages/setuptools/config/_apply_pyprojecttoml.py:196: UserWarning: `install_requires` overwritten in `pyproject.toml` (dependencies)
  warnings.warn(msg)
LGPL-2.1+
michel in pystemd on  fix-setup_cfg-deprecated-field [✘] via 🐍 v3.11.3
⬢ [fedora-toolbox:38] ❯ python3 setup.py --license
/usr/lib/python3.11/site-packages/setuptools/config/_apply_pyprojecttoml.py:196: UserWarning: `install_requires` overwritten in `pyproject.toml` (dependencies)
  warnings.warn(msg)
LGPL-2.1+
michel-slm commented 1 year ago

fix for EL build (which was because I unconditionally pushed a minor fix removing *.c unconditionally, but these don't get generated on EL) pushed, so the failed PackIt jobs can be retried.