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
fetchgit: add lfs support #105998
fetchgit: add lfs support #105998
Conversation
tried pulling a few existing packages that use |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/how-to-fetch-lfs-enabled-repo-with-fetchfromgithub/5890/15 |
Using LFS could cause a lot of downloads, not sure if we should advocate for people to also do |
|
dcf7db6
to
e6f8de6
Compare
If you are using LFS in a public repo you should plan for this. |
Yes, but exposing lfs within nixpkgs may encourage people to use it for packaging other projects (which is fine). Just don't want to bloat the nix cache with all of these large src pulls |
cc @infinisil (previous deepclone work) |
rebuilds are related to other scripts which depend on this script
|
I ran |
Motivation for this change
Add git-lfs support to
fetchgit
motivation from https://discourse.nixos.org/t/how-to-fetch-lfs-enabled-repo-with-fetchfromgithub/5890/13
tested with:
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)