-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
captus: specify megahit version number for linux builds #52888
Comments
Thanks @mptrsen, maybe you have a suggestion for this specific issue. There is a newer build for OSX than hfbae3c0_0, however the new one has compilation problems and doesn't run properly, that is why I was trying to require this specific older build. Unfortunately the build hfbae3c0_0 is always ignored and one has to install it manually (therefore I added these instructions in our README). If a specific build can't be invoked I would leave the recipe as:
Otherwise I would try:
Which would be your suggestion? Thanks, Edgardo |
@edgardomortiz thanks for your swift response! If the OSX build hfbae3c0_0 is ignored and needs to be installed manually anyway then I would simplify the recipe to
Manual installation is suboptimal, I hope the OSX users see the notice in the README. Thank you! |
Yeah, I thought so, I was hoping there was a reliable method to point to a specific build. Thanks @mptrsen |
@mptrsen I opted for the following:
The checks passed, just waiting for the merge. Thanks, Edgardo |
Without version number, this can pull in an old version of megahit that is missing critical command line options (such as
--tmp-dir
). As a result, the Captus assembly stage fails, silently.Remove the non-versioned megahit entry in
meta.yaml
and use v1.2.9 for both builds:bioconda-recipes/recipes/captus/meta.yaml
Lines 32 to 33 in d378c71
@edgardomortiz thank you! 🙏
The text was updated successfully, but these errors were encountered: