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
rush: fix build #36822
rush: fix build #36822
Conversation
Success on x86_64-linux (full log) Attempted: rush Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: rush Partial log (click to expand)
|
pkgs/shells/rush/default.nix
Outdated
patches = [ ./fix-format-security-error.patch ]; | ||
patches = [ ./fix-format-security-error.patch | ||
(fetchpatch { | ||
url = "https://sources.debian.org/data/main/r/rush/1.8+dfsg-1.1/debian/patches/intprops.patch"; |
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.
Unfortunately this is not a stable URL. When Debian releases new versions this version will go away. As the packaging it not in version control either, we should copy the patch into nixpkgs.
222634e
to
9aca6f3
Compare
Debian patch copied into nixpkgs and squashed commits |
@GrahamcOfBorg eval |
@GrahamcOfBorg eval |
@GrahamcOfBorg build rush |
Evaluation doesn't fail for me. |
Success on x86_64-linux (full log) Attempted: rush Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: rush Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: rush Partial log (click to expand)
|
Sorry, I misread the Debian bug report. Actually, rush bundles some code from a newer version of gnulib, which caused the build error with gcc7. It does not use our old version of gnulib at all... |
Motivation for this change
An incompatibility with our latest gnulib, see Debian bug 853649. Fortunately, Debian has a patch.
/cc ZHF #36453
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)