Skip to content

Commit

Permalink
Merge pull request #7772 from chrahunt/refactor/command-level-scoped-…
Browse files Browse the repository at this point in the history
…tracker

Use Command context helper to cleanup requirement tracker
  • Loading branch information
chrahunt committed Feb 23, 2020
2 parents 94c83c0 + eb91e8c commit ea47be5
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 3 deletions.
4 changes: 3 additions & 1 deletion src/pip/_internal/commands/download.py
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,9 @@ def run(self, options, args):
)
build_delete = (not (options.no_clean or options.build_dir))

with get_requirement_tracker() as req_tracker, TempDirectory(
req_tracker = self.enter_context(get_requirement_tracker())

with TempDirectory(
options.build_dir, delete=build_delete, kind="download"
) as directory:
reqs = self.get_requirements(
Expand Down
4 changes: 3 additions & 1 deletion src/pip/_internal/commands/install.py
Original file line number Diff line number Diff line change
Expand Up @@ -284,7 +284,9 @@ def run(self, options, args):
build_delete = (not (options.no_clean or options.build_dir))
wheel_cache = WheelCache(options.cache_dir, options.format_control)

with get_requirement_tracker() as req_tracker, TempDirectory(
req_tracker = self.enter_context(get_requirement_tracker())

with TempDirectory(
options.build_dir, delete=build_delete, kind="install"
) as directory:
try:
Expand Down
4 changes: 3 additions & 1 deletion src/pip/_internal/commands/wheel.py
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,9 @@ def run(self, options, args):
options.wheel_dir = normalize_path(options.wheel_dir)
ensure_dir(options.wheel_dir)

with get_requirement_tracker() as req_tracker, TempDirectory(
req_tracker = self.enter_context(get_requirement_tracker())

with TempDirectory(
options.build_dir, delete=build_delete, kind="wheel"
) as directory:
reqs = self.get_requirements(
Expand Down

0 comments on commit ea47be5

Please sign in to comment.