Conversation
At least on armv7l-linux bootstrapTools fixed version of bits/statx.h
breaks the header:
NixOS#209870 (comment)
Drop the header along with other already cleaned ones.
misuzu
left a comment
There was a problem hiding this comment.
Not sure why it works, but it works!
|
Yeah, it's a bit tricky. Here is now I understand it: What is fixincludesIn a steady state to build basic software you need a compiler (say, In the best case To work around that fact How fixincludes breaksOn What is worse What is even worse is that
Our breakage happens around The fix effectively throws away some of the invalid |
|
Reviewing now. If this works then it is a much better idea than regenerating the bootstrapFiles. |
ghost
left a comment
There was a problem hiding this comment.
Thank you so much for tracking this down, @trofi!
I confirmed this fixes pkgsCross.armv7l-hf-multiplatform.stdenv.__bootPackages.stdenv.__bootPackages.stdenv.__bootPackages.xz.
I have a few other builds running to make sure this doesn't cause problems anywhere else but I doubt that will happen.
At least on armv7l-linux bootstrapTools fixed version of bits/statx.h breaks the header:
#209870 (comment)
Drop the header along with other already cleaned ones.
Description of changes
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)