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
handbrake: move FDK codec to optional choice #38809
handbrake: move FDK codec to optional choice #38809
Conversation
I thought about what NixOS tests could I apply to package like this (C++ GTK+ package). |
@@ -75,7 +75,6 @@ stdenv.mkDerivation rec { | |||
configureFlags = [ | |||
"--disable-df-fetch" | |||
"--disable-df-verify" | |||
"--enable-fdk-aac" |
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.
Could you make this optional like useGtk
below, so that users could still enable it easily?
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 didn't knew there is such possibility.
Ok. I would redo it now.
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.
Yes, you can install it using pkgs.handbrake.override { useFdk = 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.
Oh. Yes, you are right.
I used those options, but newer implemented them until now.
@GrahamcOfBorg build handbrake |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: handbrake Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: handbrake Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: handbrake Partial log (click to expand)
|
Motivation for this change
Enabling FDK by default violates GPL2 license of HandBrake.
HandBrake 1.1.0 build fails in Nix HandBrake/HandBrake#1273 (comment)
They had skew back in time: https://forum.handbrake.fr/viewtopic.php?f=33&t=34143
This is a very serious question for them throught the years.
Nix HandBrake is marked GPL2. And as such also allowed in free NixOS space. FDK codec has a weird non-free license that is not compatible with GPL2.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)