-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Lazy trees #6530
base: master
Are you sure you want to change the base?
Lazy trees #6530
Changes from 250 commits
37d859b
a115c4f
91aea15
13c0db4
def7b25
034340a
f95c425
30be644
7da3a30
301f388
440214f
89f1021
120bec5
8a43eaa
7c1bdff
c0dd35a
2d5cfca
3667cf5
b293b33
48a5879
1b8065f
85c1959
432a3a1
2a1c63c
28b62dd
a291e37
c3c0682
cbade16
241dd54
5c8d67d
0286edb
1483c56
7317196
5115909
12dd8d4
e2353b9
0402dd0
1683872
e424a8b
a653e98
f02da62
b275aa4
8342317
4072024
64a69b4
561440b
b27cd88
515b908
39a783f
116acc1
fcdca3d
730f6bf
57397a3
d162222
6d104bb
f620184
d950e3a
12c554a
a322306
d8620d7
8becbb0
3bcaa7d
210cd8c
81a4516
880a72b
15d2e0e
b48e641
5e3cd3e
29dff7e
20a0a74
fa5af1e
9286b1a
9512afa
3621d07
3522978
f4f0f8a
37b4a9e
31bb875
aaf8b1b
b14830b
5c29abc
4142982
ca26ce9
219510b
7b1cda9
00b746d
bb421ac
12f1413
02d5c54
26ff9c1
f4009fd
ece20d5
5d1e5a0
c8f7ca2
273df09
21f0a98
03618bb
56a845d
98a90cc
430bfcf
13e8d70
23806db
ab2b0fb
8c8f242
579ecd3
1b0b3b1
834d878
1017b11
86ca2c5
e952d36
d6cdb07
334348f
733861d
3a1a33d
b19534a
854a311
b8218b5
f21e1cf
aa823c2
d7d49fd
bb72d1b
6192fd0
4f1b0d7
cdb946e
7052e19
2f1a90b
0671807
b2a205e
ccd2ad2
2a4462e
f432967
f23b969
2ffa909
4b9215c
9a15ec8
870e750
64b9e50
2154084
a0b0cac
a1501cf
8ee59d7
791e222
5b2be3a
0dfc214
91aefbd
d102273
39079dd
2b9ded5
a9d4780
2dc2f58
ac73702
3c4d678
e59caad
2f18e78
f355c34
56272db
3494c29
4e16d5e
6b70509
eaa785e
ea5c2e3
b219d76
0e48afb
0b72b31
6513f69
94c028f
e350f84
fa8618f
144987e
6a95ff9
b4df104
601de7f
c677ea7
aa85359
ddca787
ec272de
b39148e
3099b51
0aa13b6
be1543e
653c100
575902b
d38f4d9
dab2be3
2055e28
c0b1df7
748567b
5558d65
d022bce
7eb3ba0
d9cfae2
43de535
0d02a63
d2292a1
edcd6ce
793a838
1c56bdd
49f9099
ce23764
3369a81
3f70f20
1abf058
147593d
c6b4f01
cf02c6a
4a9b7c8
0efcbbf
c4cd4cd
93cf285
2fec506
5ddd8e4
1302bfa
9c10fcf
71575a2
333d35a
0950094
2101916
fc4d948
db6c219
a65c8a9
c5ae41d
ea54be0
a435a40
7332321
299d0b7
3a96ba0
25635e5
b5be3b0
2409e6b
44d20c3
4133514
2cb003c
8fc36e2
bcb1be4
f0cf5d8
a123f87
26f1431
b7255f8
584fece
4d39908
e255be6
a33aad3
35bbd85
b6be10d
57e1692
57b2cc2
1b20945
53fabdc
34e4a61
0d3a157
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -129,6 +129,7 @@ | |
bzip2 xz brotli editline | ||
openssl sqlite | ||
libarchive | ||
libgit2 | ||
boost | ||
lowdown-nix | ||
] | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Found another issue when using
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Another very general issue: There's 4 different results you can get when printing lazy paths from this PR:
Paths were already hard to understand, but this makes it even more confusing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found an issue
the
nix edit
man page saysbut that returns a virtual path instead of the path used by
nix edit
nix edit ".#hello"
works properly and is an editable pathhttps://github.com/NixOS/nixpkgs/blob/e80f82aa1edc853339ae889568cfa430808b0dfe/pkgs/stdenv/generic/make-derivation.nix#L137-L142
builtins.unsafeGetAttrPos
is used to createmeta.position
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another issue
In this filter https://github.com/serokell/nixfmt/blob/7e9e06eefed52d6d698b828ee83b83ea5c163f3b/flake.nix#L66-L68
Only the top level files and directories are passed in the path arg
applying
and doing
nix build ".#packages.x86_64-linux.nixfmt.src"
gives the following his branch
and the following on
2.17