manodeep / Corrfunc

⚡️⚡️⚡️Blazing fast correlation functions on the CPU.
https://corrfunc.readthedocs.io
MIT License
163 stars 50 forks source link

Hard-coded python in `python_bindings` Makefiles #169

Closed manodeep closed 5 years ago

manodeep commented 5 years ago

General information

Issue description

Hard-coded python in theory/python_bindings/Makefile and mocks/python_bindings/Makefile.

Expected behavior

The Makefile should respect the system variable $PYTHON, as is done by common.mk

Actual behavior

The default python will get used. That should probably be a backup option.

What have you tried so far?

sed -i '95s/python/$(PYTHON)/1' theory/python_bindings/Makefile
sed -i '48s/python/$(PYTHON)/1' mocks/python_bindings/Makefile

Minimal failing example

N/A: Compilation issue

Reported by Christopher Harris at Pawsey