Skip to content

Commit d028af9

Browse files
committed
Remove unnecessary write_delete_marker_file
InstallRequirement only checks for a delete marker file in source_dir. Since the result of ensure_build_location is set to source_dir (in ensure_has_source_dir), and ensure_build_location returns a subdirectory of the build_directory to which we write the delete marker file, then this delete marker file is never used.
1 parent 1d17df6 commit d028af9

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

src/pip/_internal/req/req_install.py

-2
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@
3636
from pip._internal.utils.marker_files import (
3737
PIP_DELETE_MARKER_FILENAME,
3838
has_delete_marker_file,
39-
write_delete_marker_file,
4039
)
4140
from pip._internal.utils.misc import (
4241
ask_path_exists,
@@ -371,7 +370,6 @@ def ensure_build_location(self, build_dir):
371370
if not os.path.exists(build_dir):
372371
logger.debug('Creating directory %s', build_dir)
373372
os.makedirs(build_dir)
374-
write_delete_marker_file(build_dir)
375373
return os.path.join(build_dir, name)
376374

377375
def _set_requirement(self):

0 commit comments

Comments
 (0)