Page MenuHomePhabricator

Update Quibble setuptools check to modern alternative
Open, LowPublic

Description

With Gerrit 540882 I introduced tox -e check to validate the package. It runs:

python setup.py --version
python setup.py check

The check is configured via:

setup.cfg
[check]
metadata = true
# Requires docutils
#restructuredtext = true
strict = true

Both commands now raises a deprecation warning about requirements that should be satisfied by a PEP 517 installer.

check: commands[1]> python setup.py check
/home/hashar/projects/integration/quibble/.tox/venv-py3/lib/python3.11/site-packages/setuptools/__init__.py:84: _DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated.
!!

        ********************************************************************************
        Requirements should be satisfied by a PEP 517 installer.
        If you are using pip, you can try `pip install --use-pep517`.
        ********************************************************************************

!!
  dist.fetch_build_eggs(dist.setup_requires)
running check

I guess we can live without output the version, but the checks should be kept around. I could not determine what setuptools check verifies exactly nor how to port those verification or whether they are sill needed.

Event Timeline

For python setup.py --version that can be replaced by:

tox.ini
[testenv:check]
deps = setuptools_scm
commands = python -m setuptools_scm

For python setup.py check I think we should move Quibble to pyproject.toml and a build system. https://packaging.python.org/en/latest/tutorials/packaging-projects/ is a good reference.

Change 958495 had a related patch set uploaded (by Hashar; author: Hashar):

[integration/quibble@master] Modernize installation

https://gerrit.wikimedia.org/r/958495

Change 958495 merged by jenkins-bot:

[integration/quibble@master] Migrate from setup.py to pyproject.toml (PEP 517)

https://gerrit.wikimedia.org/r/958495

Change 982641 had a related patch set uploaded (by Hashar; author: Hashar):

[integration/quibble@master] release: Quibble 1.6.0

https://gerrit.wikimedia.org/r/982641

Change 982641 merged by jenkins-bot:

[integration/quibble@master] release: Quibble 1.6.0

https://gerrit.wikimedia.org/r/982641

Change 982814 had a related patch set uploaded (by Hashar; author: Hashar):

[integration/config@master] dockerfiles: update Quibble to 1.6.0

https://gerrit.wikimedia.org/r/982814

Change 982814 merged by jenkins-bot:

[integration/config@master] dockerfiles: update Quibble to 1.6.0

https://gerrit.wikimedia.org/r/982814