Skip to content

Commit

Permalink
Merge pull request #2230 from xavfernandez/delete_marker_filename
Browse files Browse the repository at this point in the history
remove InstallRequirement.delete_marker_filename
  • Loading branch information
dstufft committed Dec 19, 2014
2 parents 54dd5d4 + 65c004d commit 5addd1b
Showing 1 changed file with 4 additions and 9 deletions.
13 changes: 4 additions & 9 deletions pip/req/req_install.py
Original file line number Diff line number Diff line change
Expand Up @@ -875,11 +875,11 @@ def prepend_root(path):
def remove_temporary_source(self):
"""Remove the source files from this requirement, if they are marked
for deletion"""
if os.path.exists(self.delete_marker_filename):
if self.source_dir and os.path.exists(
os.path.join(self.source_dir, PIP_DELETE_MARKER_FILENAME)):
logger.debug('Removing source in %s', self.source_dir)
if self.source_dir:
rmtree(self.source_dir)
self.source_dir = None
rmtree(self.source_dir)
self.source_dir = None
if self._temp_build_dir and os.path.exists(self._temp_build_dir):
rmtree(self._temp_build_dir)
self._temp_build_dir = None
Expand Down Expand Up @@ -988,11 +988,6 @@ def move_wheel_files(self, wheeldir, root=None):
isolated=self.isolated,
)

@property
def delete_marker_filename(self):
assert self.source_dir
return os.path.join(self.source_dir, PIP_DELETE_MARKER_FILENAME)


def _strip_postfix(req):
"""
Expand Down

0 comments on commit 5addd1b

Please sign in to comment.