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

Not recognising bcftools version >=1.5 #350

Closed
nathanhaigh opened this issue May 17, 2018 · 2 comments · Fixed by #352
Closed

Not recognising bcftools version >=1.5 #350

nathanhaigh opened this issue May 17, 2018 · 2 comments · Fixed by #352

Comments

@nathanhaigh
Copy link
Contributor

Since BCFtools 1.4.1 releases have been using a 2 digits for their versions (i.e. 1.5, 1.6, 1.7 and 1.8). As such, the code in Sambamba nolonger recognises the version string generated by BCFtools.

Unless Sambamba doesn't support BCFtools > 1.4.1 I think the offending line is:

auto r = regex(r"Version: 1\.\d\.\d[^\n]+");

I would suggest a less constrained regex and perhaps following the same path as the version detection for SAMtools:

enforce(samtoolsVersion.startsWith("Version: 1."), "version " ~ samtoolsVersion ~ " of samtools is unsupported");

@nathanhaigh
Copy link
Contributor Author

This affects Sambamba v >0.6.5.

Commit 2d02baf actually changed the version detection from the "samtools-like" version detection. Perhaps a better way is to just may the "patch" number optional in the regex.

A minimal change might

@pjotrp
Copy link
Member

pjotrp commented May 17, 2018

Good find. Will fix.

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 a pull request may close this issue.

2 participants