Commit 6e942983 by David Hotham Committed by GitHub

respect build requirements at poetry install (#7975)

fix an issue where build requirements were not respected if a build script is used and `generate-setup-file` is set to true
parent 17454be4
...@@ -9,7 +9,6 @@ from base64 import urlsafe_b64encode ...@@ -9,7 +9,6 @@ from base64 import urlsafe_b64encode
from pathlib import Path from pathlib import Path
from typing import TYPE_CHECKING from typing import TYPE_CHECKING
from poetry.core.constraints.version import Version
from poetry.core.masonry.builders.builder import Builder from poetry.core.masonry.builders.builder import Builder
from poetry.core.masonry.builders.sdist import SdistBuilder from poetry.core.masonry.builders.sdist import SdistBuilder
from poetry.core.masonry.utils.package_include import PackageInclude from poetry.core.masonry.utils.package_include import PackageInclude
...@@ -101,16 +100,7 @@ class EditableBuilder(Builder): ...@@ -101,16 +100,7 @@ class EditableBuilder(Builder):
f.write(decode(builder.build_setup())) f.write(decode(builder.build_setup()))
try: try:
if self._env.pip_version < Version.from_parts(19, 0):
pip_install(self._path, self._env, upgrade=True, editable=True) pip_install(self._path, self._env, upgrade=True, editable=True)
else:
# Temporarily rename pyproject.toml
renamed_pyproject = self._poetry.file.path.with_suffix(".tmp")
self._poetry.file.path.rename(renamed_pyproject)
try:
pip_install(self._path, self._env, upgrade=True, editable=True)
finally:
renamed_pyproject.rename(self._poetry.file.path)
finally: finally:
if not has_setup: if not has_setup:
os.remove(setup) os.remove(setup)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment