From d78e6cc8173849d29c0ed5698b08bad6d12857a2 Mon Sep 17 00:00:00 2001 From: Gilles Peskine Date: Sat, 22 Jan 2022 00:44:07 +0100 Subject: [PATCH] Any package used in a script must be listed in ci.requirements.txt This includes scripts that don't run on the CI, but are analyzed by all.sh check_python_files. Signed-off-by: Gilles Peskine --- scripts/ci.requirements.txt | 4 ++++ scripts/maintainer.requirements.txt | 3 ++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/scripts/ci.requirements.txt b/scripts/ci.requirements.txt index 18b40ec17..338b14f6e 100644 --- a/scripts/ci.requirements.txt +++ b/scripts/ci.requirements.txt @@ -1,5 +1,9 @@ # Python package requirements for Mbed TLS testing. +# Any package used by a script in this repository must be listed here +# or in one of the included files. Normally there should be a minimum +# version constraint; the CI will test with the minimum version. + # Use a known version of Pylint, because new versions tend to add warnings # that could start rejecting our code. # 2.4.4 is the version in Ubuntu 20.04. It supports Python >=3.5. diff --git a/scripts/maintainer.requirements.txt b/scripts/maintainer.requirements.txt index b149921a2..87341405e 100644 --- a/scripts/maintainer.requirements.txt +++ b/scripts/maintainer.requirements.txt @@ -1,4 +1,5 @@ -# Python packages that are only useful to Mbed TLS maintainers. +# Python packages that are not used by any script in this repository, +# but are likely to be useful to Mbed TLS maintainers. -r ci.requirements.txt