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

Sync CLI to Cargo.toml #39

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

Conversation

rhagenson
Copy link
Contributor

I implemented the changes needed for closing #36, however I am facing serious build issues with librocksdb-sys via rust-htslib on my current machine so I am going to sanity check on another machine later this evening.

@rhagenson
Copy link
Contributor Author

Update: the sanity check on my other machine resulted in a new set of problems. Original machine: problems with rocksdb; new machine: problems with linking to GSL.

I will open the PR up for review to hopefully allow someone else to verify only the intended changes were made.

@rhagenson rhagenson marked this pull request as ready for review January 30, 2020 02:45
@HenningTimm
Copy link
Contributor

I experienced a libgsl linking error when compiling from a conda environment. The path to libgsl that conda knew pointed to a different version than expected (or was otherwise not working). Unfortunately i cannot reliably reproduce and debug this problem. Some things that might help:

  • libgsl-dev needs to be installed
  • The conda package gsl needs to be installed

Maybe we should open an Issue for this.

@rhagenson
Copy link
Contributor Author

@HenningTimm, I tried both installing libgsl-dev and installing via conda. Best lead I have on what might be causing the issue is the Rust wrapper rgsl experiences issues with GSL v2, but we have the necessary feature in Cargo.toml already so not sure that is the problem. I will open an issue to provide a thread for investigating the linking problem(s).

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

Successfully merging this pull request may close these issues.

2 participants