-
Notifications
You must be signed in to change notification settings - Fork 130
Fix installation tests: compile and run icb_arpack_c. #425
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
All flavors of all pc files should now be tested. The question I still have here is: as arpack is a lib that needs other libs (blas, ...), does pc files must embed libs needed (blas, ...) by arpack? For now, pc files doesn't: to compile/link with arpack, one need to add compile/link also with libs needed (blas, ...) by arpack. Not sure what is the right thing to do. |
Now every cases are working and tested: With means If you don't "help" cmake with extra MPI target you provide, pc file is not enough... |
Should use |
|
All flavors of all pc/cmake files should now be tested. |
Pull request purpose
fixing issue #399 and related to #398 (comment)