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
Add J-Link binary package for embedded debugging #80990
Conversation
I rebased your commit on top of the master and fixed issues I've found. You might want to pull this into your master branch overwriting the current PR: prusnak@67c5c1b |
Tried building this on nixos-unstable but it fails with:
Adding |
homepage = "https://www.segger.com/downloads/jlink"; | ||
description = "SEGGER J-Link"; | ||
license = licenses.unfree; | ||
platforms = platforms.linux; |
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.
Should this list the platforms from architecture
? That is: platforms = [ "x86_64-linux" "i686-linux" "armv7l-linux" ]
.
J-Link binary package for embedded debugging
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/nixpkgs-adding-ld-library-path/11573/1 |
I would really love to have jlink supported in master. Is there anything I could do to help bringing this to master? |
sha256 = { | ||
x86_64-linux = "0vc7x6kjs92p6qdsn2lazmvlw7npz1z7r9ipj85wd123m7hgwnmg"; | ||
i686-linux = "01qm56jyac3mzjny1z5lynik8y4hqrfq93n8119mvj6d4xiknv8y"; | ||
armv7l-linux = "03l2zkfjw7z6j6nsdw6j4nxxzh8mgby8qrc179qjcajbdr3hmbr7"; |
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.
Misses the aarch64-linux
release, which has much better binary cache support.
Superseded by and incorporated into by #121601. |
|
||
dontConfigure = true; | ||
dontBuild = true; | ||
dontStrip = true; |
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.
I did not include the dontStrip
in the new version. Is that something we need?
Motivation for this change
J-Link is a common tool used in embedded development. It'd be great to be able to use it on NixOS
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)