lix-project/flake-compat
16
0
Fork 0
Code Issues 13 Pull requests Packages Projects Releases Wiki Activity

Support submodules for fetchTree #19

Closed
timothyklim wants to merge 1 commit from master into master
pull from: master
merge into: lix-project:master
Conversation 1 Commits 1 Files changed 1 +3 -2
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
default.nix
View file

@ -25,10 +25,11 @@ let
}
else if info.type == "git" then
{ outPath =
builtins.fetchGit
({ url = info.url; }
builtins.fetchTree
({ inherit (info) type url; }
// (if info ? rev then { inherit (info) rev; } else {})
edolstra commented 2022-02-25 10:24:13 +00:00 (Migrated from github.com)
Review
Copy link

We shouldn't use fetchTree because it's an experimental feature and is not available in older versions of Nix (so using it would kind of negate the point of flake-compat).

We shouldn't use `fetchTree` because it's an experimental feature and is not available in older versions of Nix (so using it would kind of negate the point of `flake-compat`).
// (if info ? ref then { inherit (info) ref; } else {})
// (if info ? submodules then { inherit (info) submodules; } else {})
);
lastModified = info.lastModified;
lastModifiedDate = formatSecondsSinceEpoch info.lastModified;