Skip to content

Commit

Permalink
Running 'pip install --upgrade setuptools' with 'sudo' on macOS, fixing
Browse files Browse the repository at this point in the history
  • Loading branch information
YannickJadoul committed Jan 30, 2019
1 parent c6f4f01 commit fe4bb3c
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions cibuildwheel/macos.py
Expand Up @@ -86,7 +86,8 @@ def call(args, env=None, cwd=None, shell=False):
# install pip & wheel
call(['python', get_pip_script, '--no-setuptools', '--no-wheel'], env=env)
call(['pip', '--version'], env=env)
call(['pip', 'install', '--upgrade', 'setuptools'], env=env)
# sudo required, because the removal of the old version of setuptools might cause problems with newer pip versions (see issue #122)
call(['sudo', 'pip', 'install', '--upgrade', 'setuptools'], env=env)
call(['pip', 'install', 'wheel'], env=env)
call(['pip', 'install', 'delocate'], env=env)

Expand Down Expand Up @@ -118,11 +119,11 @@ def call(args, env=None, cwd=None, shell=False):
delocated_wheel = glob('/tmp/delocated_wheel/*.whl')[0]

# install the wheel
call(['pip', 'install', delocated_wheel], env=env)
call(['pip', 'install', '--user', delocated_wheel], env=env)

# test the wheel
if test_requires:
call(['pip', 'install'] + test_requires, env=env)
call(['pip', 'install', '--user'] + test_requires, env=env)
if test_command:
# run the tests from $HOME, with an absolute path in the command
# (this ensures that Python runs the tests against the installed wheel
Expand Down

0 comments on commit fe4bb3c

Please sign in to comment.