Skip to content

Commit

Permalink
Merge pull request #185005 from amjoseph-nixpkgs/pr/fetchgit/fix-comment
Browse files Browse the repository at this point in the history
  • Loading branch information
Artturin authored Jan 14, 2023
2 parents b591671 + c8a1838 commit 1121c8f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkgs/build-support/fetchgit/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ in

/* NOTE:
fetchgit has one problem: git fetch only works for refs.
This is because fetching arbitrary (maybe dangling) commits may be a security risk
This is because fetching arbitrary (maybe dangling) commits creates garbage collection risks
and checking whether a commit belongs to a ref is expensive. This may
change in the future when some caching is added to git (?)
Usually refs are either tags (refs/tags/*) or branches (refs/heads/*)
Expand Down

0 comments on commit 1121c8f

Please sign in to comment.