Skip to content

Fetch Git LFS objects#1355

Open
hacker1024 wants to merge 1 commit intoNixOS:masterfrom
hacker1024:git-lfs
Open

Fetch Git LFS objects#1355
hacker1024 wants to merge 1 commit intoNixOS:masterfrom
hacker1024:git-lfs

Conversation

@hacker1024
Copy link
Member

@hacker1024 hacker1024 commented Jan 31, 2024

This PR changes the Git input plugin to fetch LFS objects alongside the regular git fetch when creating SCM caches. This allows the objects to be used in subsequent clones from the cache (done automatically when programs.git-lfs.enable is true).

This is harmless for repositories that do not use LFS.

Related: #824 (I'm not sure how this affects flakes).

@DerDennisOP
Copy link

@NixOS/hydra-maintainers could someone please merge this?

@Daholli
Copy link
Member

Daholli commented May 5, 2025

I just came across this very issue, I recently started building my system configs with hydra

https://hydra.christophhollizeck.dev/build/192#tabs-summary
this build succeeded and using nix-serve I presumably got everything I needed, but when I rebuilt the system both images I have in my lfs that are put in the nix store using homemanagers file.source were missing but referenced

image
and since my system believes that it got everything properly from the cache it does not rebuild even tho there are files missing in the store

gitattributes

*.png filter=lfs diff=lfs merge=lfs -text
*.webp filter=lfs diff=lfs merge=lfs -text
*.jpg filter=lfs diff=lfs merge=lfs -text

how i put the files into the nix store

home.file = {
  "Pictures/firewatch.jpg".source = ./firewatch.jpg;
};

Daholli added a commit to Daholli/nixos-config that referenced this pull request May 6, 2025
Daholli added a commit to Daholli/nixos-config that referenced this pull request May 6, 2025
Daholli added a commit to Daholli/nixos-config that referenced this pull request May 6, 2025
@dasJ dasJ added the evaluator hydra-evaluator component label Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

evaluator hydra-evaluator component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants