Move tsv-utils from bioconda, build from source rather than binary repackage

conda-forge / staged-recipes

A place to submit conda recipes before they become fully fledged conda-forge feedstocks

https://conda-forge.org

BSD 3-Clause "New" or "Revised" License

701 stars 4.88k forks source link

Move tsv-utils from bioconda, build from source rather than binary repackage #26778

Closed corneliusroemer closed 3 months ago

corneliusroemer commented 3 months ago

Checklist

[x] Title of this PR is meaningful: e.g. "Adding my_nifty_package", not "updated meta.yaml".
[x] License file is packaged (see here for an example).
[x] Source is from official source.
[x] Package does not vendor other packages. (If a package uses the source of another package, they should be separate packages or the licenses of all packages need to be packaged).
[x] If static libraries are linked in, the license of the static library is packaged.
[x] Package does not ship static libraries. If static libraries are needed, follow CFEP-18.
[x] Build number is 0.
[x] A tarball (url) rather than a repo (e.g. git_url) is used in your recipe (see here for more details).
[x] GitHub users listed in the maintainer section have posted a comment confirming they are willing to be listed there.
[x] When in trouble, please check our knowledge base documentation before pinging a team.

conda-forge-webservices[bot] commented 3 months ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipes/tsv-utils) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipes/tsv-utils:

Recipe with the same name exists in bioconda: please discuss with @conda-forge/bioconda-recipes.
This recipe is using a compiler, which now requires adding a build dependence on {{ stdlib("c") }} as well. Note that this rule applies to each output of the recipe using a compiler. For further details, please see https://github.com/conda-forge/conda-forge.github.io/issues/2102.

conda-forge-webservices[bot] commented 3 months ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipes/tsv-utils) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipes/tsv-utils:

Recipe with the same name exists in bioconda: please discuss with @conda-forge/bioconda-recipes.

corneliusroemer commented 3 months ago

Intending to move tsv-utils from bioconda to conda-forge, as it's not really a bioinfo specific tool, but of general utility. Pinging @conda-forge/bioconda-recipes for discussion

corneliusroemer commented 3 months ago

PR to remove tsv-utils from bioconda, to be merged once this PR here is merged: https://github.com/bioconda/bioconda-recipes/pull/48813