Skip to content
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

Separate git repo for mpq library? #6

Open
smoe opened this issue Aug 6, 2018 · 1 comment
Open

Separate git repo for mpq library? #6

smoe opened this issue Aug 6, 2018 · 1 comment

Comments

@smoe
Copy link

smoe commented Aug 6, 2018

Hello,

while compiling hhsuite with openmpi installed, I ran into an error because of a missing mpq library that you ship with ffindex. I then quickly deinstalled openmpi, but the better approach may be to offer the header file and the library to link against. Would you accept a patch against ffindex to create respective library? Or is there another git repository for mpq out there somewhere? Should there be one? In #4 I already learned that you treat
ffindex as a git subproject, which completely explains why you don't run into any such issue on your side since you have everything in one tree. But the users of yours who compile from your hhsuite release source tree will notice since ffindex is then external to that tree, which is just the right way how it should be for the redistribution. Just, somehow, mpq is a bit lost.

Cheers,

Steffen

@ZivBA
Copy link

ZivBA commented Jan 3, 2021

I confirm this is still happening.
Any chance of adding the MPQ library to hhsuite tree as well?

  • Correction - this only happens when trying to compile on a slurm cluster node, possibly due to mismatched openmpi versions.

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

No branches or pull requests

2 participants