Skip to content

Commit

Permalink
Add FIXME: to all workaround comments
Browse files Browse the repository at this point in the history
  • Loading branch information
TheLastProject committed Mar 25, 2017
1 parent 41d5b33 commit e7ba456
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions pext/__main__.py
Original file line number Diff line number Diff line change
Expand Up @@ -485,7 +485,7 @@ def _pip_install(self, module_dir_name: str) -> int:
'pip',
'install']

# Cheap hack to work around Debian's faultily-patched pip
# FIXME: Cheap hack to work around Debian's faultily-patched pip
if os.path.isfile('/usr/bin/aptitude'):
pip_command += ['--system']

Expand All @@ -497,7 +497,7 @@ def _pip_install(self, module_dir_name: str) -> int:

returncode = 0

# Cheap macOS workaround, part 1
# FIXME: Cheap macOS workaround, part 1
# See https://github.com/pypa/pip/pull/4111#issuecomment-280616124
if platform.system() == "Darwin":
with open(os.path.expanduser('~/.pydistutils.cfg'), 'w') as macos_workaround:
Expand All @@ -509,7 +509,7 @@ def _pip_install(self, module_dir_name: str) -> int:
except CalledProcessError as e:
returncode = e.returncode

# Cheap macOS workaround, part 2
# FIXME: Cheap macOS workaround, part 2
if platform.system() == "Darwin":
os.remove(os.path.expanduser('~/.pydistutils.cfg'))

Expand Down

0 comments on commit e7ba456

Please sign in to comment.