-
Notifications
You must be signed in to change notification settings - Fork 431
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
Deprecate conda_build.bdist_conda
#4985
Comments
A few thoughts:
|
I'd be interested to hearing what @dholth has to say about this |
I would be shocked if anyone uses this. |
@mbargull do you have any thoughts or concerns about deprecating |
Thanks for asking, Bianca! In general, I think it would be beneficial if If we leave it in There is, of course, the questions on where such tool/package would be developed at and who would sponsor such work... |
I'm arriving to this issue, investigating after my CI pipeline has started failing with
realizing that it's been deprecated, and reading:
Well, for what is worth, at least one person was using it 🥲. Also my project didn't appear in that search (and probably others as well) because I don't have Of course I understand the need of deprecating features that are barely used, but be careful with your assumptions: there could be more people using this than what you expect. |
BTW if there are others like me, for me the solution is just to fix the version to |
What happened?
The functionality in
conda_build.bdist_conda
is specific to Python packages (built withdistutils
/setuptools
) and not needed to build a conda package, e.g. see the sample usage:conda-build/recipe/test_bdist_conda_setup.py
Lines 3 to 11 in b2cb091
Based on a preliminary GitHub search the feature is also minimally used out in the wild (if at all?).
Docs
Thoughts on deprecating it? @chenghlee @dbast @jaimergp @ryanskeith @beeankha
The text was updated successfully, but these errors were encountered: