-
-
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
simplify flake #6538
simplify flake #6538
Conversation
c6cc7f4
to
5d53ef3
Compare
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.
Thank you!!
@edolstra any objections to merge? |
Yes please merge this. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/2022-12-16-nix-team-meeting-minutes-17/24120/1 |
The issue *seems* to be the cross jobs, which are missing the `CXXFLAGS` needed to get rapidcheck. PR NixOS#6538 would be really nice to resurrect which will prevent the `configureFlags` from going out of sync between the regular build and the cross build again.
The issue *seems* to be the cross jobs, which are missing the `CXXFLAGS` needed to get rapidcheck. PR NixOS#6538 would be really nice to resurrect which will prevent the `configureFlags` from going out of sync between the regular build and the cross build again.
The issue *seems* to be the cross jobs, which are missing the `CXXFLAGS` needed to get rapidcheck. PR NixOS#6538 would be really nice to resurrect which will prevent the `configureFlags` from going out of sync between the regular build and the cross build again.
I think this PR should be assigned to a single person who takes full ownership. |
The merge conflicts are why I haven't been rebasing it anymore. I was waiting to get a clear signal that this would get merged fairly quickly after. So please tell me "as the Nix team ..." and then I will re-do it. |
I think we moved it to "in review" in triage without posting a thing here in hopes someone would fix it. I will move back to in discussion so we can get @zimbatm the "clear signal". |
Or alternatively someone on the @nixos/nix-team maybe feels comfortable approving the concept here async. |
@zimbatm The team has approved this change. I will review and merge it as soon as possible. |
(sorry hit wrong button!) |
@zimbatm |
5d53ef3
to
01b74e2
Compare
fa164ac
to
943a0f5
Compare
ok, this is ready for a round of review. I will squash the two commits afterwards. |
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.
It looks good now! Further cleanups / de-duping can be done in follow-up PRs.
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.
Checked by evaluating some of the attributes. No changes in the default package or alternate stdenv packages. Small improvements to the cross package, such as separate debug info.
d() { nix-diff "$(nix eval ".?ref=upstream/master#$1" | jq -r .)" "$(nix eval ".#$1" | jq -r .)"; }
Note to self: have a look at the hydra comparison after this is merged. |
couldn't resist to add two more noop changes :) |
Lovely! Could you squash the fixups? |
86f5d81
to
98d9ad5
Compare
done (there was one merge conflict in the manual) |
caused by f2e4279 |
98d9ad5
to
5d7d69c
Compare
- `nixpkgsFor` does all of native, static, cross, and the different stdenvs. - The main Nix derivation is no longer duplicated for static. - DRY nixpkgs.lib and lib.genAttrs calls.
5d7d69c
to
d7a4f08
Compare
fixed |
Some dependencies supposed to be skipped in the cross build, along with not using the gold linker. But in NixOS#6538 this was accidentally not preserved. Also since NixOS#6538 we saw some new aarch64-linux static build failures. This is a first attempt to try to fix those failures. If this is not sufficient, there are other things we can try next.
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/nix-team-report-2022-10-2023-03/27486/1 |
Basically finishing #6196
/cc @Ericson2314