Skip to content

Commit

Permalink
build: use pyproject.toml to comply with pep517
Browse files Browse the repository at this point in the history
Also: remove --no-index for installing ourselves.

PEP-517 compliance means we need to use PyPI to get setuptools, so we
can't use --no-index anymore.  I think this was added just to speed up
the initial installation of the virtualenv used in these tests.
  • Loading branch information
nedbat committed Sep 13, 2022
1 parent c4ad568 commit 447334c
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
6 changes: 6 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
# Licensed under the Apache License: http://www.apache.org/licenses/LICENSE-2.0
# For details: https://github.com/nedbat/coveragepy/blob/master/NOTICE.txt

[build-system]
requires = ['setuptools', 'wheel']
build-backend = 'setuptools.build_meta'
2 changes: 1 addition & 1 deletion tests/test_venv.py
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ def testp():

# Install everything.
run_in_venv(
"python -m pip install --no-index " +
"python -m pip install " +
"./third_pkg " +
"-e ./another_pkg " +
"-e ./bug888/app -e ./bug888/plugin " +
Expand Down

0 comments on commit 447334c

Please sign in to comment.