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
r2-cutter: init at 1.1(.0), radare2 qt gui #33198
Conversation
(Soft ping 😇) |
let tag = "1.1"; in | ||
stdenv.mkDerivation rec { | ||
name = "r2-cutter-${version}"; | ||
version = "${tag}.0"; |
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.
Why do you add .0
to the version number?
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.
Hmm, don't remember. Something worked better with "full" version specification, maybe some tool I use. If it seems odd I'll change it :).
pkgs/top-level/all-packages.nix
Outdated
@@ -7707,6 +7707,7 @@ with pkgs; | |||
rubyBindings = config.radare.rubyBindings or false; | |||
luaBindings = config.radare.luaBindings or false; | |||
}; | |||
r2-cutter = qt5.callPackage ../development/tools/analysis/cutter { }; |
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.
Please use libsForQt5
.
AUR names it radare2-cutter
(https://aur.archlinux.org/packages/radare2-cutter/), would you like that name? I'd also rename the directory to radare2-cutter
for consistency.
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.
Aye aye.
radare2 qt gui, using name used in AUR since 'cutter' already exists.
562c830
to
bcf81b8
Compare
Thanks for review @orivej! Updated to address your comments. |
@GrahamcOfBorg build radare2-cutter |
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Failure on aarch64-linux (full log) Partial log (click to expand)
|
@dtzWill any idea about the aarch64 failure? If not, perhaps constrain the platform list to only x86 linux? |
Looks to be another instance of NixOS/aarch64-build-box#17, although this is less clear than other cases. |
(FWIW: 1.2 is out, but has a few bugs so waiting until 1.3 or whatever is next) |
Naming "r2-cutter" since "cutter" already exists,
but AFAIK this naming is not used anywhere else currently.
Motivation for this change
Updated version of #32345 for 1.1 release (and new radare2 in #33194)
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)