Skip to content
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

uhd: apply upstream patch to fix build w/boost 1.66 #33552

Closed
wants to merge 1 commit into from

Conversation

dtzWill
Copy link
Member

@dtzWill dtzWill commented Jan 7, 2018

Fixes #33549.

Motivation for this change

Grab upstream patch for fixing compat with boost 1.66.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@adisbladis
Copy link
Member

@GrahamcOfBorg build uhd

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Failure for system: x86_64-darwin

--:
--:-
-
 0:00
:01
3667
9

cannot build derivation ‘/nix/store/ala10w98gr0as7jzkpgbnnnsslryj6hp-uhd-3.10.2.0.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/ala10w98gr0as7jzkpgbnnnsslryj6hp-uhd-3.10.2.0.drv’ failed

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Success for system: aarch64-linux

uhd-images_003.007.003-release/share/uhd/images/usrp_n200_r2_fpga.bin
uhd-images_003.007.003-release/share/uhd/images/usrp_b200_fw.hex
uhd-images_003.007.003-release/share/uhd/images/usrp1_fpga.rbf
uhd-images_003.007.003-release/share/uhd/images/usrp_b100_fpga.bin
uhd-images_003.007.003-release/share/uhd/images/003.007.003.tag
uhd-images_003.007.003-release/share/uhd/images/usrp_x310_fpga_HGS.lvbitx
uhd-images_003.007.003-release/share/uhd/images/usrp_n210_r2_fpga.bin
uhd-images_003.007.003-release/share/uhd/images/usrp_x300_fpga_HGS.lvbitx
uhd-images_003.007.003-release/share/uhd/images/usrp_n200_r4_fpga.bin
/nix/store/9hv37si96cr6334vvmvxdh5z7pf8icvg-uhd-3.10.2.0

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Success for system: x86_64-linux

uhd-images_003.007.003-release/share/uhd/images/usrp_n200_r2_fpga.bin
uhd-images_003.007.003-release/share/uhd/images/usrp_b200_fw.hex
uhd-images_003.007.003-release/share/uhd/images/usrp1_fpga.rbf
uhd-images_003.007.003-release/share/uhd/images/usrp_b100_fpga.bin
uhd-images_003.007.003-release/share/uhd/images/003.007.003.tag
uhd-images_003.007.003-release/share/uhd/images/usrp_x310_fpga_HGS.lvbitx
uhd-images_003.007.003-release/share/uhd/images/usrp_n210_r2_fpga.bin
uhd-images_003.007.003-release/share/uhd/images/usrp_x300_fpga_HGS.lvbitx
uhd-images_003.007.003-release/share/uhd/images/usrp_n200_r4_fpga.bin
/nix/store/c7zq8jhazwpn4zb3a5r9ya95dgasg9hs-uhd-3.10.2.0

@peterhoeg peterhoeg mentioned this pull request Jan 7, 2018
8 tasks
@peterhoeg
Copy link
Member

@dtzWill is this tested and good to go?

@dtzWill
Copy link
Member Author

dtzWill commented Jan 7, 2018

I was responding to the issue/build failure, which this fixes. But I'm not sure how to test the resulting derivation "works"-- although since this is a minor change from upstream I expect it works as well as it did previously.

@markuskowa
Copy link
Member

Thanks for fixing it. Unfortunately, I can't test it either. I opened #33549 since uhd is needed by gnuradio.

@lukeadams
Copy link
Contributor

lukeadams commented Jan 10, 2018

@dtzWill
Copy link
Member Author

dtzWill commented Jan 16, 2018

Closing (this PR seems to be wrong approach) in favor of updating to 3.10.3.0 when it's released.

Should be soon... :)

@dtzWill dtzWill closed this Jan 16, 2018
@lukeadams
Copy link
Contributor

@markuskowa markuskowa mentioned this pull request Jan 30, 2018
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Package uhd fails to build
6 participants