Skip to content

Commit b7c7dd2

Browse files
committed
Fixed pymodules install - it works now
1 parent 1c27f31 commit b7c7dd2

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

pythonforandroid/toolchain.py

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -342,7 +342,6 @@ def get_env(self):
342342
toolchain_prefix=toolchain_prefix,
343343
cxxflags=env['CXXFLAGS'])
344344

345-
# AND: Not sure if these are still important
346345
env['AR'] = '{}-ar'.format(toolchain_prefix)
347346
env['RANLIB'] = '{}-ranlib'.format(toolchain_prefix)
348347
env['LD'] = '{}-ld'.format(toolchain_prefix)
@@ -1910,14 +1909,13 @@ def run_pymodules_install(ctx, modules):
19101909
fileh.write('{}\n'.format(module))
19111910

19121911
info('Installing Python modules with pip')
1912+
info('If this fails with a message about /bin/false, this '
1913+
'probably means the package cannot be installed with '
1914+
'pip as it needs a compilation recipe.')
19131915

1914-
# AND: This doesn't work yet
1915-
shprint(sh.bash, '-c', '''"source venv/bin/activate && env CC=/bin/false CXX=/bin/false PYTHONPATH= pip install --target '{}' -r requirements.txt"'''.format(ctx.get_site_packages_dir()))
1916+
shprint(sh.bash, '-c', '''source venv/bin/activate && env CC=/bin/false CXX=/bin/false PYTHONPATH= pip install --target '{}' -r requirements.txt'''.format(ctx.get_site_packages_dir()))
1917+
exit(1)
19161918

1917-
1918-
1919-
1920-
19211919
def biglink(ctx, arch):
19221920
# First, collate object files from each recipe
19231921
info('Collating object files from each recipe')

0 commit comments

Comments
 (0)