saopicc / killMS

Package for direction dependent calibration
GNU General Public License v2.0
10 stars 3 forks source link

Make the Makefile work everywhere #2

Closed nudomarinero closed 7 years ago

nudomarinero commented 7 years ago

Take the right includes independently of the system. It works as long as the Python headers and numpy are installed.

mhardcastle commented 7 years ago

Looks sensible -- not that the -I to non-existent directories should cause any problem, but we might as well get rid of them in the public version. Will merge (and worry about propagation upstream later).

nudomarinero commented 7 years ago

Actually, the main problem is that the -I did not cover all the possible locations of the Python libraries. It failed on a clean installation in which the numpy headers were in the standard /usr/local/lib/python2.7/dist-packages/numpy/core/include/numpy. With the new code the compiler gets the right paths always.