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
ed: fix docs location. Fixes #32150. Supersedes #32315. #32322
Conversation
by removing installFlags and configureFlags. Fixes NixOS#32159
Thanks! Fixed the issues you outlined. |
The lack of the CC will probably break cross though. |
After @orivej suggested it could be omitted, I confirmed that it made no difference on my system, so we removed it. |
Cross exports CC, and |
I'd love to be wrong, but I recall the configure script was not influenced by the environment, falling back on its own definition of |
https://hydra.nixos.org/eval/1416753 yeah, |
#32489 fixes cross |
Motivation for this change
DESTDIR
was set manually, but otherinstallFlags
were missing. Nix could fill the gaps, but that resulted in some wrong directories: man pages ended up in/nix/store/<hash>-ed/nix/store/<hash>-ed/share/
.The solution is simple: let nix figure it all out.
#32315 caused mass rebuilding. As suggested by @orivej, I rebased the commit on
staging
.I now also removed the configureFlags.
Please review the entire default.nix file for further improvements. I 'd be happy to include more 'modernizations' and avoid extra mass rebuilding.
There's no maintainer set. Any volonteers?
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)