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
blackmagic: v1.6.1-317-gc9c8b08 -> v1.6.1-409-g7a595ea (add new targets) #79026
Conversation
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.
LGTM. Sorry for not getting around to this myself; I was hoping there'd be a tagged upstream release sooner.
@GrahamcOfBorg build blackmagic
@GrahamcOfBorg build blackmagic |
Looks like stdenv is passing down CFLAGS it shouldn't to arm-embedded-gcc on Darwin. If you're not sure how to fix that, probably best just to mark it as only building on Linux for now? |
fyi ZHF: #80379 |
Hopefully the bump will fix the error, but whoever merges should make sure to switch platforms to Linux-only so they don't introduce a new one in the process ^^; |
I'll take a look and update to more recent version as well. |
Add myself to maintainers and change meta.platforms to platforms.linux due to build failures on Darwin.
Bumped version, now uses |
@GrahamcOfBorg build blackmagic |
(cherry picked from commit 855f020) It didn't build before this bump.
Motivation for this change
Update to more recent version. Also exposes hosted targets which can be run on host system instead of microcontroller (requires libusb and pkg-config).
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)