-
Notifications
You must be signed in to change notification settings - Fork 4
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
Cool with custom channel anaconda release? #152
Comments
A few quick thoughts -
thank you for checking! |
Thanks! We’ll wait to hear from @luizirber before pursuing a bioconda release in a custom channel. Looks like the most straightforward way in the time-being is a post-install pip script. We’ll use that until a full conda approach becomes feasible and/or you all release branchwater to conda |
seeing if maybe we can just build a conda-forge recipe for branchwater - trial run here |
Yup, that was going to be my suggestion! Footnotes
|
whoo! linux build worked! |
hi @dkoslicki I put through the rename and got the Linux build working. Unfortunately, I'm out of time to work on it intensively today (and probably tomorrow). SO, if you & yours would like to help that would be lovely! The two thoughts I had -
Probably the easiest would be to just submit PRs into my branch, the source branch here, and I can merge them quickly - I have time today, just not big chunks. |
The linux-only build seems to have worked. Shall I put the PR up for merge or do we want to try to get OS X to work first? @luizirber @dkoslicki @bluegenes |
We've been having a heck of a time getting OS X to work, mainly due to bioconda moving to Azure, it looks like their build system is missing a necessary clang lib to compile Rust stuff. I'd vote just go with linux for the time being as we pursue the missing lib issue in the bioconda Gitter channel. |
conda-forge/staged-recipes#24530 merged! I'll close this issue (and open a new one about |
Whoot! I just noticed it's officially available on conda-forge now. Thanks @ctb ! |
Yep, and looks like it all works!
Let us know if you run into any problems building on top of this - excited to see YACHT out there! |
closing in favor of #160 for future work! |
so @luizirber got sourmash-minimal working on conda-forge, and I copied his approach in conda-forge/sourmash_plugin_branchwater-feedstock#3 - so both sourmash-minimal and branchwater should be working on MacOS now! |
Awesome, thanks for the update! That looked rather nontrivial to do! |
Hi @ctb , @bluegenes and others! We'd (@chunyuma) like to publish YACHT to conda and it uses a specific commit of branchwater as a dependency. However, conda doesn't like that the way we're currently using pip, and it was recommended to us to make a conda release of branchwater in a custom channel. Are y'all ok with this? We'd make sure to clearly indicate the brancheater in this custom channel is meant only for installing YACHT.
The text was updated successfully, but these errors were encountered: