Skip to content

Update for Python 3.12: check_python_dependencies.py (GIT8266O-891) #1314

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wstanzl wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update for Python 3.12: check_python_dependencies.py (GIT8266O-891) #1314

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
49 changes: 44 additions & 5 deletions tools/check_python_dependencies.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,14 +18,53 @@
import os
import sys

print(f"Using python {sys.version} located at {sys.executable}")

try:
from importlib.metadata import version, PackageNotFoundError
except Exception as e:
print(str(e))
print('version, PackageNotFoundError from importlib.metadata cannot be imported probably because the pip package is not installed and/or using a '
'legacy Python interpreter. Please refer to the Get Started section of the ESP-IDF Programming Guide for '
'setting up the required packages. Also note that this version of check_python_dependencies.py has migrated '
'from using pkg_resources to importlib and packaging since pkg_resources has been removed from python.')
sys.exit(1)

try:
from packaging.requirements import Requirement
except Exception as e:
print(str(e))
print('Requirement from packaging.requirements cannot be imported probably because the pip package is not installed and/or using a '
'legacy Python interpreter. Please refer to the Get Started section of the ESP-IDF Programming Guide for '
'setting up the required packages. Also note that this version of check_python_dependencies.py has migrated '
'from using pkg_resources to importlib and packaging since pkg_resources has been removed from python.')
sys.exit(1)

try:
import pkg_resources
except Exception:
print('pkg_resources cannot be imported probably because the pip package is not installed and/or using a '
from packaging.specifiers import SpecifierSet
except Exception as e:
print(str(e))
print('SpecifierSet from packaging.specifiers cannot be imported probably because the pip package is not installed and/or using a '
'legacy Python interpreter. Please refer to the Get Started section of the ESP-IDF Programming Guide for '
'setting up the required packages.')
'setting up the required packages. Also note that this version of check_python_dependencies.py has migrated '
'from using pkg_resources to importlib and packaging since pkg_resources has been removed from python.')
sys.exit(1)

def require(requirement_line: str):
if requirement_line != '#':
return void

req = Requirement(requirement_line)
try:
installed_version = version(req.name)
except PackageNotFoundError:
print(f"Package {req.name} is not installed")
sys.exit(1)

if installed_version not in req.specifier:
print(f"{req.name}{req.specifier} required, but {installed_version} is installed")
sys.exit(1)


def escape_backslash(path):
if sys.platform == "win32":
Expand Down Expand Up @@ -56,7 +95,7 @@ def is_virtualenv():
for line in f:
line = line.strip()
try:
pkg_resources.require(line)
require(line)
except Exception:
not_satisfied.append(line)

Expand Down