Bug 208334

Summary: multimedia/handbrake: Update to 0.10.5
Product: Ports & Packages Reporter: Ben Woods <woodsb02>
Component: Individual Port(s)Assignee: Thomas Zander <riggs>
Status: Closed FIXED    
Severity: Affects Only Me CC: riggs
Priority: --- Keywords: needs-qa, patch
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://handbrake.fr/news.php?article=36
Attachments:
Description Flags
Patch to update multimedia/handbrake to 0.10.5
none
QA: Successful poudriere testport logs for multimedia/handbrake 0.10.5 with patch on FreeBSD 11-current amd64 (gzipped)
none
QA: Successful poudriere testport logs for multimedia/handbrake 0.10.5 with patch on FreeBSD 9.3 i386 (gzipped)
none
Revised patch for X11 OPTION
none
Revised patch including FDK_AAC OPTION none

Description Ben Woods freebsd_committer freebsd_triage 2016-03-27 17:10:47 UTC 
Created attachment 168693 [details]
Patch to update multimedia/handbrake to 0.10.5

* Update multimedia/handbrake to 0.10.5
* Remove FDK AAC encoder, due to upstream license dispute [1]

  [1] https://forum.handbrake.fr/viewtopic.php?f=33&t=34143

  Changes:
    https://handbrake.fr/news.php?article=36
Comment 1 Ben Woods freebsd_committer freebsd_triage 2016-03-27 19:33:41 UTC 
Created attachment 168701 [details]
QA: Successful poudriere testport logs for multimedia/handbrake 0.10.5 with patch on FreeBSD 11-current amd64 (gzipped)
Comment 2 Ben Woods freebsd_committer freebsd_triage 2016-03-27 19:34:14 UTC 
Created attachment 168702 [details]
QA: Successful poudriere testport logs for multimedia/handbrake 0.10.5 with patch on FreeBSD 9.3 i386 (gzipped)
Comment 3 Thomas Zander freebsd_committer freebsd_triage 2016-03-28 14:29:19 UTC 
Created attachment 168729 [details]
Revised patch for X11 OPTION

Ben, your first patch does not build successfully without the X11 OPTION. This adds an ugly hack to make it compile.

In any case, I would like to turn fdk-aac into a non-default OPTION. For people building the port themselves and accepting license consequences, we should not just take away the best aac codec we currently have.
Comment 4 Thomas Zander freebsd_committer freebsd_triage 2016-04-23 16:07:00 UTC 
Pushed a new patch to phabricator for review:

https://reviews.freebsd.org/D6077

Could you test it?
Comment 5 Thomas Zander freebsd_committer freebsd_triage 2016-04-23 16:10:03 UTC 
Created attachment 169604 [details]
Revised patch including FDK_AAC OPTION

Patch is also available on
https://reviews.freebsd.org/D6077
for review
Comment 6 commit-hook freebsd_committer freebsd_triage 2016-04-24 15:12:25 UTC 
A commit references this bug:

Author: riggs
Date: Sun Apr 24 15:11:56 UTC 2016
New revision: 413944
URL: https://svnweb.freebsd.org/changeset/ports/413944

Log:
  Update to upstream version 0.10.5, turning FDK_AAC into an OPTION

  PR:		208334
  Submitted by:	woodsb02@gmail.com
  Reviewed by:	riggs, jbeich
  Differential Revision:	https://reviews.freebsd.org/D6077

Changes:
  head/multimedia/handbrake/Makefile
  head/multimedia/handbrake/distinfo