nbench: supply stdenv.glibc.static since Makefile uses -static#172173
Conversation
nbench's Makefile always compiles with -static, so we need it to be able to see libc.a. Let's add stdenv.glibc.static to the buildInputs.
|
The ZHF announcement PR says: "Please ping @NixOS/nixos-release-managers on the PR and add the 0.kind: build failure label to the pull request. If you're unable to because you're not a member of the NixOS org please ping @dasJ, @tomberek, @jonringer, @Mic92" I am unable to add this build label, so I therefore am issuing this ping. |
|
Please squash the commits. |
|
I squashed them in the github UI. |
|
We usually create merge commits which are missing with this, that's why i haven't used it. |
I massively prefer squashing, it keeps the history readable and cherry-pickable (you can't Git merges were never meant to be used the way the github web UI uses them -- for applying patchsets. Git merges were always meant for long-lived branches that get merged more than once, like our Github pushes excessive use of merge commits because they don't like the fact that rebasing/cherrypicking a commit changes its hash. This costs them more storage (less deduplication), but the real problem is that it breaks all their big data analytics. |
|
I agree, but this is not the place to change the rules. |
Oh, I didn't know there was a formal policy about this. I've seen people merge my PRs using both styles. In any event, I should try to remember to squash commits myself. |
|
It's even worse. There is an informal policy. |
ZHF: #172160
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/)nixos/doc/manual/md-to-db.shto update generated release notesnbench's Makefile always compiles with -static, so we need it to be
able to see libc.a. Let's add stdenv.glibc.static to the buildInputs.