-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
development/libraries/glibc: work around glibc tuple-parsing bug #163401
Conversation
Glibc's configuration machinery cannot parse *-muslabi64, and fails with a very cryptic error message. This commit adds a --host=mips64el-linux-musl to extraConfigureFlags (so it comes after the --host= added by stdenv) in this case.
Nevermind, I figured it out, have to set This should no longer cause any rebuilds whatsoever. In light of that, I have changed the base branch to |
Also, I am confused why we are building glibc when we are trying to use musl! |
Will take a look. |
It was somewhere within
... I'm rebuilding with this patch reverted to find the exact reason. Yeah, I recall being surprised by that too. Note that you need to be cross-compiling a static binary. Although this sounds esoteric it is incredibly useful for producing the initial I also think it would be nice if Hydra built static |
On the tip of master the only thing that didn't build for me was boost $ nix-build . -A pkgs.pkgsCross.mips64el-linux-gnuabi64.pkgsStatic.nix_2_4 --dry-run
these derivations will be built:
/nix/store/gjayd0z6742dp30ynpcqs757kkdin7a7-boost-static-mips64el-unknown-linux-muslabi64-1.77.0.drv
/nix/store/d7c5p1xin7x6b98pn78i0hgzakzqrd9l-nix-static-mips64el-unknown-linux-muslabi64-2.4.drv I think that means we can close this. |
Yes. Tracking down exactly what was going on here was harder than I thought, so let me document it. For the record: this problem was introduced in the early revisions of my PR #161158. @Ericson2314 pointed out here that I was missing cases in the "convert this platform to a statically-linked platform" function. That function had no name at the time, and was copy-pasted into multiple different places; it has since been refactored as
Anyways, if this happens again, we know why. |
@Ericson2314 by the way, that is fixed by #161162, which has had no objections since it was opened almost two months ago (just two minor formatting requests, both implemented)... |
Glibc's configuration machinery cannot parse *-muslabi64, and fails with a very cryptic error message. This commit adds a --host=mips64el-linux-musl to extraConfigureFlags (so it comes after the --host= added by stdenv) in this case.
This depends on #161158 and was split off from it as requested here
Description of changes
Things done
sandbox = true
set 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.sh
to update generated release notes