On Fri, Feb 26, 2021 at 10:31 PM Tom Rini <[email protected]> wrote: > > On Fri, Feb 26, 2021 at 10:25:30PM +0800, Bin Meng wrote: > > On Fri, Feb 26, 2021 at 8:53 PM Tom Rini <[email protected]> wrote: > > > > > > Given that test/py/requirements.txt has all required test modules, make > > > use of that rather than a manual pip install list before running our > > > assorted tool testsuites. > > > > > > Signed-off-by: Tom Rini <[email protected]> > > > --- > > > .azure-pipelines.yml | 2 +- > > > .gitlab-ci.yml | 2 +- > > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml > > > index a6279427e138..0a4f90f75f3b 100644 > > > --- a/.azure-pipelines.yml > > > +++ b/.azure-pipelines.yml > > > @@ -144,7 +144,7 @@ jobs: > > > export USER=azure > > > virtualenv -p /usr/bin/python3 /tmp/venv > > > . /tmp/venv/bin/activate > > > - pip install pyelftools pytest pygit2 > > > + pip install -r test/py/requirements.txt > > > > It seems only pytest is listed in test/py/requirements.txt? > > Not since: > commit f4a60b036308ca4a0a3ad08587b7f9b72d3565d5 > Author: Igor Opaniuk <[email protected]> > Date: Mon Jan 25 14:28:41 2021 +0200 > > test: py: add pygit2 and pyelftools to requirements.txt
Ah, yes! Was looking at an old branch. Reviewed-by: Bin Meng <[email protected]>
