mirror of
https://github.com/espressif/esp-idf.git
synced 2025-08-10 12:53:29 +00:00

The `setuptools` package starting with `v70.1.0`[1] contains built-in
`bdist_wheel` command. Before this version `setuptools` relied on the
`bdist_wheel` command implementation from the `wheel` package. Starting with
`setuptools` `v75.8.1` the `PEP 491`[3] restrictions on the distribution name
of a wheel package are enforced[4], replacting also `.` with `_`. Note that
`PEP 491` actually allows `.` in the distribution name, but for some reason the
latest packaging docs[10][11] does not, stating that `.` should be replaced
with `_`. This was discussion here[12].
Also the `wheel` package starting with `v0.45.0`[5] is using the `bdist_wheel`
command from `setuptools`. This means that any package which has `.` in its
distribution name, like `ruamel.yaml.clib`, can have different wheel name,
depending on which version of the `bdist_wheel` command was used.
The `bdist_wheel` command from setuptools prior `v75.8.1` or `wheel` prior
`v0.45.0` will keep the dots in distribution name preserved. For exaple the
`ruamel.yaml.clib` package will have distribution name
`ruamel.yaml.clib-0.2.12.dist-info. Newer versions will replace the dots with
`_` according to [10][11], creating distribution like
`ruamel_yaml_clib-0.2.12.dist-info`.
From packaging point of view `ruamel.yaml.clib-0.2.12.dist-info` and
`ruamel_yaml_clib-0.2.12.dist-info` are the same packages, but this is not
reflected in `importlib.metadata` prior python 3.10[9], which does not perform
name normalization prior the distribution search. This causes the `version`
from `importlib.metadata` to fail on python prior the 3.10 version if the
package with dots in distribution name was generated with normalized paths with
newer `setuptools`. Note that the distribution name normalization was
backported to some later 3.9 python version.
Let's demonstrate this behavior on a simple package with the
`my.minimal.package` name.
```
my_minimal_package/
├── pkg
│ └── __init__.py
└── setup.py
from setuptools import setup, find_packages
setup(
name='my.minimal.package',
version='0.1.0',
packages=find_packages(),
install_requires=[],
entry_points={},
)
```
With python 3.9.0 search for `my.minimal.package` fails because
of the missing name normalization.
```
docker run --rm -it --platform linux/x86_64 python:3.9.0 bash
python -m venv venv
. venv/bin/activate
pip install setuptools==v75.8.1
python setup.py bdist_wheel
pip install dist/my_minimal_package-0.1.0-py3-none-any.whl
python
Python 3.9.0 (default, Nov 18 2020, 13:28:38)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from importlib.metadata import version as get_version
>>> get_version('my.minimal.package')
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/usr/local/lib/python3.9/importlib/metadata.py", line 551, in version
return distribution(distribution_name).version
File "/usr/local/lib/python3.9/importlib/metadata.py", line 524, in distribution
return Distribution.from_name(distribution_name)
File "/usr/local/lib/python3.9/importlib/metadata.py", line 187, in from_name
raise PackageNotFoundError(name)
importlib.metadata.PackageNotFoundError: my.minimal.package
>>> get_version('my_minimal_package')
'0.1.0'
```
With python 3.10.0 search for both `my.minimal.package` and
`my_minimal_package` succeeds.
```
docker run --rm -it --platform linux/x86_64 python:3.10.0 bash
python
Python 3.10.0 (default, Dec 3 2021, 00:21:30) [GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from importlib.metadata import version as get_version
>>> get_version('my.minimal.package')
'0.1.0'
>>> get_version('my_minimal_package')
'0.1.0'
```
In our `tools/check_python_dependencies.py` we cannot relay on the default
distribution finder, used in the `version` function from `importlib.metadata`,
to do name normalization on older python versions. To cope with this,
implement a fallback version search. If `version` fails with
`PackageNotFoundError`, do the name normalization according to [10][11] and try
again.
Note: There is also a `wheel`[6][7] `v0.43.0` package embeded in `setuptools`
along with the new implementation[8]. This one seems to be used if the
external `wheel` package is not available but imported. TBH this is all kinda
messy and I may have overlooked something.
* [1] https://setuptools.pypa.io/en/stable/history.html#v70-1-0
* [2] https://setuptools.pypa.io/en/stable/history.html#v75-8-1
* [3] https://peps.python.org/pep-0491/#escaping-and-unicode
* [4] https://github.com/pypa/setuptools/pull/4766/files
* [5] https://wheel.readthedocs.io/en/stable/news.html
* [6] https://github.com/pypa/setuptools/blob/main/setuptools/_vendor/wheel/__init__.py
* [7] https://github.com/pypa/setuptools/issues/1386
* [8] https://github.com/pypa/setuptools/blob/main/setuptools/command/bdist_wheel.py
* [9] c6ca368867
* [10] https://packaging.python.org/en/latest/specifications/name-normalization/#name-normalization
* [11] https://packaging.python.org/en/latest/specifications/binary-distribution-format/
#escaping-and-unicode
* [12] https://github.com/pypa/setuptools/issues/3777
Signed-off-by: Frantisek Hrbata <frantisek.hrbata@espressif.com>
182 lines
8.9 KiB
Python
Executable File
182 lines
8.9 KiB
Python
Executable File
#!/usr/bin/env python
|
|
#
|
|
# SPDX-FileCopyrightText: 2018-2025 Espressif Systems (Shanghai) CO LTD
|
|
# SPDX-License-Identifier: Apache-2.0
|
|
import argparse
|
|
import os
|
|
import re
|
|
import sys
|
|
from typing import Optional
|
|
|
|
try:
|
|
from packaging.requirements import Requirement
|
|
from packaging.version import Version
|
|
except ImportError:
|
|
print('packaging cannot be imported. '
|
|
'If you\'ve installed a custom Python then this package is provided separately and have to be installed as well. '
|
|
'Please refer to the Get Started section of the ESP-IDF Programming Guide for setting up the required packages.')
|
|
sys.exit(1)
|
|
|
|
try:
|
|
from importlib.metadata import requires as _requires
|
|
from importlib.metadata import version as _version
|
|
from importlib.metadata import PackageNotFoundError
|
|
except ImportError:
|
|
# compatibility for python <=3.7
|
|
from importlib_metadata import requires as _requires # type: ignore
|
|
from importlib_metadata import version as _version # type: ignore
|
|
from importlib_metadata import PackageNotFoundError # type: ignore
|
|
|
|
try:
|
|
from typing import Set
|
|
except ImportError:
|
|
# This is a script run during the early phase of setting up the environment. So try to avoid failure caused by
|
|
# Python version incompatibility. The supported Python version is checked elsewhere.
|
|
pass
|
|
|
|
PYTHON_PACKAGE_RE = re.compile(r'[^<>=~]+')
|
|
|
|
|
|
# The version and requires function from importlib.metadata in python prior
|
|
# 3.10 does perform distribution name normalization before searching for
|
|
# package distribution. This might cause problems for package with dot in its
|
|
# name as the wheel build backend(e.g. setuptools >= 75.8.1), may perform
|
|
# distribution name normalization. If the package name is not found, try again
|
|
# with normalized name.
|
|
# https://packaging.python.org/en/latest/specifications/binary-distribution-format/#escaping-and-unicode
|
|
def normalize_name(name: str) -> str:
|
|
return re.sub(r'[-_.]+', '-', name).lower().replace('-', '_')
|
|
|
|
|
|
def get_version(name: str) -> str:
|
|
try:
|
|
version = _version(name)
|
|
except PackageNotFoundError:
|
|
version = _version(normalize_name(name))
|
|
return version
|
|
|
|
|
|
def get_requires(name: str) -> Optional[list]:
|
|
try:
|
|
requires = _requires(name)
|
|
except PackageNotFoundError:
|
|
requires = _requires(normalize_name(name))
|
|
return requires
|
|
|
|
|
|
if __name__ == '__main__':
|
|
parser = argparse.ArgumentParser(description='ESP-IDF Python package dependency checker')
|
|
parser.add_argument('--requirements', '-r',
|
|
help='Path to a requirements file (can be used multiple times)',
|
|
action='append', default=[])
|
|
parser.add_argument('--constraints', '-c', default=[],
|
|
help='Path to a constraints file (can be used multiple times)',
|
|
action='append')
|
|
args = parser.parse_args()
|
|
|
|
required_set = set()
|
|
for req_path in args.requirements:
|
|
with open(req_path, encoding='utf-8') as f:
|
|
required_set |= set(i for i in map(str.strip, f.readlines()) if len(i) > 0 and not i.startswith('#'))
|
|
|
|
constr_dict = {} # for example package_name -> package_name==1.0
|
|
for const_path in args.constraints:
|
|
with open(const_path, encoding='utf-8') as f:
|
|
for con in [i for i in map(str.strip, f.readlines()) if len(i) > 0 and not i.startswith('#')]:
|
|
if con.startswith('file://'):
|
|
con = os.path.basename(con)
|
|
elif con.startswith('--only-binary'):
|
|
continue
|
|
elif con.startswith('-e') and '#egg=' in con: # version control URLs, take the egg= part at the end only
|
|
con_m = re.search(r'#egg=([^\s]+)', con)
|
|
if not con_m:
|
|
print('Malformed input. Cannot find name in {}'.format(con))
|
|
sys.exit(1)
|
|
con = con_m[1]
|
|
|
|
name_m = PYTHON_PACKAGE_RE.search(con)
|
|
if not name_m:
|
|
print('Malformed input. Cannot find name in {}'.format(con))
|
|
sys.exit(1)
|
|
constr_dict[name_m[0]] = con.partition(' #')[0] # remove comments
|
|
|
|
not_satisfied = [] # in string form which will be printed
|
|
|
|
# already_checked set is used in order to avoid circular checks which would cause looping.
|
|
already_checked = set() # type: Set[Requirement]
|
|
|
|
# required_set contains package names in string form without version constraints. If the package has a constraint
|
|
# specification (package name + version requirement) then use that instead. new_req_list is used to store
|
|
# requirements to be checked on each level of breath-first-search of the package dependency tree. The initial
|
|
# version is the direct dependencies deduced from the requirements arguments of the script.
|
|
new_req_list = [Requirement(constr_dict.get(i, i)) for i in required_set]
|
|
|
|
def version_check(requirement: Requirement) -> None:
|
|
# compare installed version with required
|
|
version = Version(get_version(requirement.name))
|
|
if not requirement.specifier.contains(version, prereleases=True):
|
|
not_satisfied.append(f"Requirement '{requirement}' was not met. Installed version: {version}")
|
|
|
|
# evaluate markers and check versions of direct requirements
|
|
for req in new_req_list[:]:
|
|
if not req.marker or req.marker.evaluate():
|
|
try:
|
|
version_check(req)
|
|
except Exception as e:
|
|
# Catch general exception, because get_version may return None (https://github.com/python/cpython/issues/91216)
|
|
# log package name alongside the error message for easier debugging
|
|
not_satisfied.append(f"Error while checking requirement '{req}'. Package was not found and is required by the application: {e}")
|
|
new_req_list.remove(req)
|
|
else:
|
|
new_req_list.remove(req)
|
|
|
|
while new_req_list:
|
|
req_list = new_req_list
|
|
new_req_list = []
|
|
already_checked.update(req_list)
|
|
for requirement in req_list: # check one level of the dependency tree
|
|
try:
|
|
dependency_requirements = set()
|
|
extras = list(requirement.extras) or ['']
|
|
# `requires` returns all sub-requirements including all extras - we need to filter out just required ones
|
|
for name in get_requires(requirement.name) or []:
|
|
sub_req = Requirement(name)
|
|
# check extras e.g. esptool[hsm]
|
|
for extra in extras:
|
|
# evaluate markers if present
|
|
if not sub_req.marker or sub_req.marker.evaluate(environment={'extra': extra}):
|
|
dependency_requirements.add(sub_req)
|
|
version_check(sub_req)
|
|
# dependency_requirements are the direct dependencies of "requirement". They belong to the next level
|
|
# of the dependency tree. They will be checked only if they haven't been already. Note that the
|
|
# version is taken into account as well because packages can have different requirements for a given
|
|
# Python package. The dependencies need to be checked for all of them because they can be different.
|
|
new_req_list.extend(dependency_requirements - already_checked)
|
|
except Exception as e:
|
|
# Catch general exception, because get_version may return None (https://github.com/python/cpython/issues/91216)
|
|
# log package name alongside the error message for easier debugging
|
|
not_satisfied.append(f"Error while checking requirement '{req}'. Package was not found and is required by the application: {e}")
|
|
|
|
if len(not_satisfied) > 0:
|
|
print('The following Python requirements are not satisfied:')
|
|
print(os.linesep.join(not_satisfied))
|
|
if 'IDF_PYTHON_ENV_PATH' in os.environ:
|
|
# We are running inside a private virtual environment under IDF_TOOLS_PATH,
|
|
# ask the user to run install.bat again.
|
|
install_script = 'install.bat' if sys.platform == 'win32' else 'install.sh'
|
|
print('To install the missing packages, please run "{}"'.format(install_script))
|
|
else:
|
|
print('Please follow the instructions found in the "Set up the tools" section of '
|
|
'ESP-IDF Getting Started Guide.')
|
|
|
|
print('Diagnostic information:')
|
|
idf_python_env_path = os.environ.get('IDF_PYTHON_ENV_PATH')
|
|
print(' IDF_PYTHON_ENV_PATH: {}'.format(idf_python_env_path or '(not set)'))
|
|
print(' Python interpreter used: {}'.format(sys.executable))
|
|
if not idf_python_env_path or idf_python_env_path not in sys.executable:
|
|
print(' Warning: python interpreter not running from IDF_PYTHON_ENV_PATH')
|
|
print(' PATH: {}'.format(os.getenv('PATH')))
|
|
sys.exit(1)
|
|
|
|
print('Python requirements are satisfied.')
|