Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Search for liblapack_atlas #341

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Search for liblapack_atlas #341

wants to merge 1 commit into from

Conversation

ktns
Copy link

@ktns ktns commented Mar 22, 2015

On a debian/ubuntu box, clapack_xgetrf is in liblapack_atlas.so.

@translunar
Copy link
Member

This is going to be a tricky one to merge, because we have to test it on a whole bunch of systems and can't just rely on Travis-CI. One thing you could do to make the process move a little faster is test it on a machine running Yosemite.

@ktns
Copy link
Author

ktns commented Mar 26, 2015

Sorry, I don't have immediate access to a MacOSX box, but I will test this patch on MacOSX soon.
By the way, I think we can safely call have_library on a system without the library.
What is your concern?

@translunar
Copy link
Member

@ktns Have a look at the commit history for extconf.rb to get an idea for how complicated it has been to make this config work for all systems. =/

@v0dro
Copy link
Member

v0dro commented Jan 13, 2016

@ktns could you please rebase again the current master and check if tests pass using the latest travis config? It should test for all systems.

@translunar
Copy link
Member

Bump.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants