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
whipper: replace morituri #46609
whipper: replace morituri #46609
Conversation
Forgot to add |
Success on x86_64-linux (full log) Attempted: accuraterip-checksum, whipper Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: accuraterip-checksum, whipper Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: accuraterip-checksum, whipper Partial log (click to expand)
|
''; | ||
|
||
meta = with stdenv.lib; { | ||
description = "A C99 commandline program to compute the AccurateRip checksum of singletrack WAV files"; |
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 think this description could be simplified to "Computes the AccurateRip checksum of singletrack WAV files".
pkgs/top-level/all-packages.nix
Outdated
@@ -3613,7 +3615,7 @@ with pkgs; | |||
|
|||
mmv = callPackage ../tools/misc/mmv { }; | |||
|
|||
morituri = callPackage ../applications/audio/morituri { }; | |||
whipper = callPackage ../applications/audio/whipper { }; |
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 think whipper
should be placed further down the list to maintain the "pseudo-alphabetic" ordering of the file.
Thanks @jtojnar, looks good to me. I just added a few minor comments. |
morituri has been dead for a while now and uses gst-python which is no longer supported wth Python 2. whipper is a maintained fork, packaged, for example, in Arch.
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: accuraterip-checksum, whipper Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: accuraterip-checksum, whipper Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: accuraterip-checksum, whipper Partial log (click to expand)
|
Nice! Merged to master 👍 |
Motivation for this change
morituri has been dead for a while now and uses gst-python which is no longer supported wth Python 2. whipper is a maintained fork, packaged, for example, in Arch.
cc @rycee @jgeerds @jansol
Closes: #27228
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)