Merge pull request #263 from sarugaku/bugfix/262

Fix PEP 517 requires
sarugaku · Aug 12, 2020 · 2d62643 · 2d62643
2 parents c05332e + 99ffef6
commit 2d62643
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 3 deletions.
diff --git a/news/262.bugfix.rst b/news/262.bugfix.rst
@@ -0,0 +1 @@
+Fix the PEP 517 requires in default ``pyproject.toml`` and clean the temp files.
diff --git a/src/requirementslib/models/setup_info.py b/src/requirementslib/models/setup_info.py
@@ -1580,8 +1580,14 @@ def pep517_config(self):
 
     def build_wheel(self):
         # type: () -> S
+        need_delete = False
         if not self.pyproject.exists():
-            build_requires = ", ".join(['"{0}"'.format(r) for r in self.build_requires])
+            if not self.build_requires:
+                build_requires = '"setuptools", "wheel"'
+            else:
+                build_requires = ", ".join(
+                    ['"{0}"'.format(r) for r in self.build_requires]
+                )
             self.pyproject.write_text(
                 six.text_type(
                     """
@@ -1593,16 +1599,21 @@ def build_wheel(self):
                     ).strip()
                 )
             )
-        return build_pep517(
+            need_delete = True
+        result = build_pep517(
             self.base_dir,
             self.extra_kwargs["build_dir"],
             config_settings=self.pep517_config,
             dist_type="wheel",
         )
+        if need_delete:
+            self.pyproject.unlink()
+        return result
 
     # noinspection PyPackageRequirements
     def build_sdist(self):
         # type: () -> S
+        need_delete = False
         if not self.pyproject.exists():
             if not self.build_requires:
                 build_requires = '"setuptools", "wheel"'
@@ -1621,12 +1632,16 @@ def build_sdist(self):
                     ).strip()
                 )
             )
-        return build_pep517(
+            need_delete = True
+        result = build_pep517(
             self.base_dir,
             self.extra_kwargs["build_dir"],
             config_settings=self.pep517_config,
             dist_type="sdist",
         )
+        if need_delete:
+            self.pyproject.unlink()
+        return result
 
     def build(self):
         # type: () -> "SetupInfo"