Remove unused RequirementSet.reqs_to_cleanup

chrahunt · chrahunt · commit 85ab574dc1e6 · 2020-02-05T20:16:05.000-05:00
diff --git a/src/pip/_internal/legacy_resolve.py b/src/pip/_internal/legacy_resolve.py
@@ -327,9 +327,6 @@ def _resolve_one(
 
         req_to_install.prepared = True
 
-        # register tmp src for cleanup in case something goes wrong
-        requirement_set.reqs_to_cleanup.append(req_to_install)
-
         abstract_dist = self._get_abstract_dist_for(req_to_install)
 
         # Parse and return dependencies
diff --git a/src/pip/_internal/req/req_set.py b/src/pip/_internal/req/req_set.py
@@ -33,7 +33,6 @@ def __init__(self, check_supported_wheels=True):
 
         self.unnamed_requirements = []  # type: List[InstallRequirement]
         self.successfully_downloaded = []  # type: List[InstallRequirement]
-        self.reqs_to_cleanup = []  # type: List[InstallRequirement]
 
     def __str__(self):
         # type: () -> str
@@ -161,7 +160,6 @@ def add_requirement(
             )
         )
         if does_not_satisfy_constraint:
-            self.reqs_to_cleanup.append(install_req)
             raise InstallationError(
                 "Could not satisfy constraints for '{}': "
                 "installation from path or url cannot be "

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,6 @@ def __init__(self, check_supported_wheels=True):`
`33`	`33`
`34`	`34`	`self.unnamed_requirements = [] # type: List[InstallRequirement]`
`35`	`35`	`self.successfully_downloaded = [] # type: List[InstallRequirement]`
`36`		`- self.reqs_to_cleanup = [] # type: List[InstallRequirement]`
`37`	`36`
`38`	`37`	`def __str__(self):`
`39`	`38`	`# type: () -> str`
`@@ -161,7 +160,6 @@ def add_requirement(`
`161`	`160`	`)`
`162`	`161`	`)`
`163`	`162`	`if does_not_satisfy_constraint:`
`164`		`- self.reqs_to_cleanup.append(install_req)`
`165`	`163`	`raise InstallationError(`
`166`	`164`	`"Could not satisfy constraints for '{}': "`
`167`	`165`	`"installation from path or url cannot be "`