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
gcc: Refactor treatment of configure flags #31292
Conversation
I still need to test this but putting it up to make sure that the direction is acceptable. I can do the same for gcc 7 in due course. |
This is great! |
Certainly 👍 for the direction. |
@nixborg build |
Jobset created at https://hydra.mayflower.de/jobset/nixos/pr-31292 |
I'm always happy to see more in the vein of #15799 :) |
How should I interpret the Travis failure? It looks to me like it just failed due to a conflict. |
@bgamari Travis fails for us, especially with mass rebuilds, so I wouldn't worry. The nixborg builds failures look spurious, so if you want to convert the other GCCs I think this is ready to go! |
dfe13ee
to
8ebf384
Compare
8ebf384
to
9e84b55
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.
Alright, I've rebased this and finished porting the remaining compiler versions.
[ | ||
"CC_FOR_BUILD=${buildPackages.stdenv.cc.prefix}gcc" | ||
"CXX_FOR_BUILD=${buildPackages.stdenv.cc.prefix}g++" | ||
] ++ |
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.
Whoops, this shouldn't be in this commit.
f4f9674
to
b9286de
Compare
a9b0d56
to
a0f2023
Compare
Previously configureFlags was defined as one giant interpolated string. Here we refactor this definition to instead use the usual stdenv string combinators. This seems more in-line with the average nixpkgs expression and it seems a bit more natural to things of these as lists of flags rather than monolithic strings.
a0f2023
to
1c12072
Compare
@nixborg build |
Jobset created at https://hydra.mayflower.de/jobset/nixos/pr-31292 |
Build looks good enough to me, thanks @bgamari! |
Previously configureFlags was defined as one giant interpolated string. Here we
refactor this definition to instead use the usual stdenv string
combinators. This seems more in-line with the average nixpkgs expression
and it seems a bit more natural to things of these as lists of flags
rather than monolithic strings.
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)