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: 3.7.11.1 -> 3.7.13.3 #41431
Conversation
# => python wrappers | ||
# May be able to upgrade to swig3 | ||
, python, swig2, numpy, scipy, matplotlib | ||
# => grc - the gnu radio companion | ||
, cheetah, pygtk | ||
, Mako, cheetah, pygtk # Note: GR is migrating to Mako. Cheetah shoul dbe removed for GR3.8 |
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.
Typo s/shoul d/should /
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.
Oops. Fixed!
@@ -26,7 +26,13 @@ stdenv.mkDerivation rec { | |||
grep -q "Requires:" "$pcfile" && { echo "Upstream has added 'Requires:' in $(basename "$pcfile"); update nix expression."; exit 1; } | |||
echo "Requires: libusb-1.0" >> "$pcfile" | |||
''; | |||
|
|||
patches = lib.optionals stdenv.isDarwin [ |
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.
Would it be possible/pertinent to open an upstream issue?
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.
Considering that the version didn't change, I assume the error is due to some Nix oddity. I can dig into it a bit more
* fixes build on Darwin. * fetch images from GH releases page (uniform versioning scheme)
* also add mako input (cheetah will be removed in GR3.8)
@matthewbauer It seems the changes I made to the src fetch broke the GR build. I'm also seeing the cmake/clang argument list issue again on master. I'll open a pr to fix the src but until the cmake -L explosion is fixed, I'll mark it as WIP since it won't build. Sorry about that :/ |
Motivation for this change
nox-review completed successfully
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)