Skip to content

Commit a01aeeb

Browse files
committed
lib/modules: Add false positive info to check coherence assertion
Unfortunately, we don't have a good enough way to discern this possibility. Technically we could add yet another attribute, but that does not seem to be worth the cost at this point, in terms of complexity and possibly even performance. I take the blame for this one. I identified the need for the second merge commit in review but didn't require it in the first merge. These changes really should have been done in a single merge. That would have prevented this situation.
1 parent 83d73e9 commit a01aeeb

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

lib/modules.nix

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1147,6 +1147,9 @@ let
11471147
instead of:
11481148
11491149
baseType // { check = value: /* your check */; }
1150+
1151+
Alternatively, this message may also occur as false positive when mixing Nixpkgs
1152+
versions, if one Nixpkgs is between 83fed2e6..58696117 (Aug 28 - Oct 28 2025)
11501153
'';
11511154

11521155
# Merge definitions of a value of a given type.

0 commit comments

Comments
 (0)