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
gnuradio: fix build for gcc6 #29740
gnuradio: fix build for gcc6 #29740
Conversation
Was gnuradio broken before? https://hydra.nixos.org/eval/1395695?filter=gnuradio&compare=1395588&full= |
oops! I was looking at the wrong branch https://hydra.nixos.org/eval/1395575?filter=gnuradio&compare=1395259&full= |
Have removed references to 17.09 release. |
Any chance this could be submitted upstream? |
@lukeadams I have raised an issue upstream gnuradio/gnuradio#1470 |
@@ -41,6 +41,7 @@ stdenv.mkDerivation rec { | |||
|
|||
enableParallelBuilding = true; | |||
|
|||
patches = [ ./gnuradio-gcc7.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.
Is there an external source for the patch? If possible, fetch it instead of checking it into the repository.
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.
The patch came from the Arch Linux packages git repo.
I have changed this PR to not use a patch, but instead apply its essential parts as NIX_CFLAGS_COMPILE
and substituteInPlace
.
@@ -41,8 +41,14 @@ stdenv.mkDerivation rec { | |||
|
|||
enableParallelBuilding = true; | |||
|
|||
patchPhase = '' |
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.
Great. It's generally better to use postPatch
instead of overriding the entire patch phase, on the off change that somebody wanted to easily apply other patches by overrding patches
.
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.
OK, updated
(cherry picked from commit 3766637)
Patch comes via Arch Linux via Fedora. It should hopefully fix the build for gcc7 (in future) as well as gcc6.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)