search

jakeret / hope

HOPE: A Python Just-In-Time compiler for astrophysical computations
GNU General Public License v3.0
382 stars 27 forks source link

Linux compiler support 3 #5

Closed iankronquist closed 9 years ago

iankronquist commented 9 years ago

Fixes issue #3

cosmo-ethz commented 9 years ago

I’m not entirely familiar with the naming schemes of compiler on Linux systems. However, to me the name looks a bit weird on Ubuntu i.e. the compiler name contains the arch and the os.

Not sure if simply adding this compiler label to the CXX_FLAGS map is the right way to go on a loner term. This could result in a large map with lots duplicated key entries. Maybe extending the compiler detection (e.g. stripping of the arch and os) might be better.

cosmo-ethz commented 9 years ago

I implementent a slightly more generic compiler detection and fixed another issue arising on Linux systems. The changes are now merged into master.