conda-build
conda-build copied to clipboard
Only fetch lfs files for specific git_ref
Description
Currently conda-build fetches all lfs files that ever existed in the history. This is not necessary, and as outlined in https://github.com/conda/conda-build/issues/4809 is costly when e.g. using GitHub. For the specific use case that I have in mind (see https://github.com/RoboStack/ros-humble/pull/155), the lfs files were actually removed from the repository and moved elsewhere (as the GitHub limits were exceeded, and now require payment), but the conda build still fails.
We already have the git_ref
that we are interested in, so this is a relatively minor change; much smaller than making git-lfs completely optional as suggested in https://github.com/conda/conda-build/issues/4809
Checklist - did you ...
- [x] Add a file to the
news
directory (using the template) for the next release's release notes? - [x] Add / update necessary tests?
- [x] Add / update outdated documentation?
We require contributors to sign our Contributor License Agreement and we don't have one on file for @Tobias-Fischer.
In order for us to review and merge your code, please e-sign the Contributor License Agreement PDF. We then need to manually verify your signature, merge the PR (https://github.com/conda/infrastructure/pull/881), and ping the bot to refresh the PR.
@conda-bot check
Hi @kenodegard - any chance to run this again, please?
I think this is ready to go - all CI failures are unrelated :)
A quick reminder @kenodegard :)
@Tobias-Fischer thanks for checking in, we're still working to resolve the unrelated test failures caused by upstream deprecations, once we untangle that we'll get this moving again
CodSpeed Performance Report
Merging #5202 will not alter performance
Comparing Tobias-Fischer:patch-1
(5a66529) with main
(ab137d2)
Summary
✅ 3
untouched benchmarks