From f154aa32fc1d94eb6f564b120b284f4ae13e38e3 Mon Sep 17 00:00:00 2001 From: Anton-Latukha Date: Thu, 10 May 2018 11:14:04 +0300 Subject: [PATCH] handbrake: updated notes --- pkgs/applications/video/handbrake/default.nix | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/pkgs/applications/video/handbrake/default.nix b/pkgs/applications/video/handbrake/default.nix index d13995f6e9bd..9f1dfb2d6b07 100644 --- a/pkgs/applications/video/handbrake/default.nix +++ b/pkgs/applications/video/handbrake/default.nix @@ -1,13 +1,9 @@ -# Handbrake normally uses its own copies of the libraries it uses, for better -# control over library patches. +# Upstream distributes HandBrake with bundle of according versions of libraries and patches to them. # -# This derivation patches HB so it doesn't do that. The relevant patches -# are added to the Nix packages and proposed upstream instead. In several cases -# upstream already incorporated these patches. -# This has the benefits of providing improvements to other packages, -# making licenses more clear and reducing compile time/install size. +# Derivation patches HandBrake to use our closure. # -# Only tested on Linux + +# TODO: Release 1.2.0 would switch LibAV to FFmpeg. { stdenv, lib, fetchurl, python2, pkgconfig, yasm, harfbuzz, zlib, @@ -34,7 +30,7 @@ stdenv.mkDerivation rec { }; patched_libav_12 = libav_12.overrideAttrs (super: { - # 2018-04-26: HandBrake compilation (1.1.0) requires a patch of LibAV (12.3) from HandBrake team. This patch not went LibAV upstream. + # NOTE: 2018-04-26: HandBrake compilation (1.1.0) requires a patch of LibAV (12.3) from HandBrake team. This patch not went LibAV upstream. patches = (super.patches or []) ++ [( fetchurl { url = ''https://raw.githubusercontent.com/HandBrake/HandBrake/9e1f245708a157231c427c0ef9b91729d59a30e1/contrib/ffmpeg/A21-mp4-sdtp.patch''; @@ -107,7 +103,6 @@ stdenv.mkDerivation rec { ''; license = licenses.gpl2; maintainers = with maintainers; [ Anton-Latukha wmertens ]; - # Not tested on anything else platforms = platforms.linux; }; }