-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
crackle: init at unstable-2020-12-13 #124172
Conversation
Result of 1 package built:
|
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.
Looks good to me. No real blockers.
|
||
stdenv.mkDerivation { | ||
pname = "crackle"; | ||
version = "unstable-2020-12-13"; |
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.
Let's see if can get a proper release: mikeryan/crackle#50
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.
No release was made so far.
|
||
src = fetchFromGitHub { | ||
owner = "mikeryan"; | ||
repo = "crackle"; |
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.
Opinionated change
repo = "crackle"; | |
repo = pname; |
Result of 1 package built:
|
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.
- package path fits guidelines
- package name fits guidelines
- package version fits guidelines
-
meta.description
is set and fits guidelines -
meta.license
fits upstream license -
meta.maintainers
is set - package built (see crackle: init at unstable-2020-12-13 #124172 (comment))
- all depending packages build
- tested on x86_64
...ew/pr-124172]$ ./results/crackle/bin/crackle
Usage: crackle -i <input.pcap> [-o <output.pcap>] [-l <ltk>] [-s]
Cracks Bluetooth Low Energy encryption (AKA Bluetooth Smart)
Major modes: Crack TK // Decrypt with LTK
[...]
Looks good to me.
Motivation for this change
#81418
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)