Bug 227374

Summary: [UPDATE] multimedia/handbrake to 1.1.0
Product: Ports & Packages Reporter: Yuichiro NAITO <naito.yuichiro>
Component: Individual Port(s)Assignee: Fernando Apesteguía <fernape>
Status: Closed FIXED    
Severity: Affects Some People CC: fernape
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
handbrake-1.1.0.patch
none
handbrake-1.1.0_without_openjdk.patch none

Description Yuichiro NAITO 2018-04-08 15:08:33 UTC
Created attachment 192341 [details]
handbrake-1.1.0.patch

Handbrake has been updated to 1.1.0
This is a patch of multimedia/handbrake port.

Fortunately most of patches have been merged to upstream.
The left patches are i386 support.

Unfortunately new libbluray requires jni.h at compile time.
So, this patch provides openjdk at build time.
It's really annoying to me but I don't have any idea better than this for now.
I think I should do as same as multimedia/libbluray,
but libbluray is extracted in HandBrake's Makefile not FreeBSD port Makefile.
I need to find how to execute extra command in HandBrake's Makefile.

Since this patch, preview works with gstreamer1.

My poudriere build has passed on following arch/versions.

i386  FreeBSD-10.4R
amd64 FreeBSD-10.4R
i386  FreeBSD-11.1R
amd64 FreeBSD-11.1R

If this patch seems reasonable,
let me take maintainership of this port.
Comment 1 Yuichiro NAITO 2018-04-10 14:09:49 UTC
Created attachment 192388 [details]
handbrake-1.1.0_without_openjdk.patch
Comment 2 Yuichiro NAITO 2018-04-10 14:10:34 UTC
I added ad hoc patch to omit openjdk.
Please check `handbrake-1.1.0_without_openjdk.patch`.

It works with both gcc6 and gcc7.
Comment 3 Fernando Apesteguía freebsd_committer freebsd_triage 2018-04-10 16:44:12 UTC
I'll take this.
Comment 4 commit-hook freebsd_committer freebsd_triage 2018-04-18 21:33:27 UTC
A commit references this bug:

Author: fernape
Date: Wed Apr 18 21:32:35 UTC 2018
New revision: 467738
URL: https://svnweb.freebsd.org/changeset/ports/467738

Log:
  Update multimedia/handbrake to 1.1.0

  Handing over maintainership to naito.yuichiro@gmail.com

  PR:	227374
  Reported by:	naito.yuichiro@gmail.com
  Approved by:	tcberner (mentor)
  Differential Revision:	https://reviews.freebsd.org/D15063

Changes:
  head/multimedia/handbrake/Makefile
  head/multimedia/handbrake/distinfo
  head/multimedia/handbrake/files/patch-contrib_libbluray_A02-expose-clip__id.patch
  head/multimedia/handbrake/files/patch-contrib_libbluray_P01-freebsd-configure.patch
  head/multimedia/handbrake/files/patch-contrib_x265__10bit_module.defs
  head/multimedia/handbrake/files/patch-contrib_x265__12bit_module.defs
  head/multimedia/handbrake/files/patch-gtk_configure.ac
  head/multimedia/handbrake/files/patch-gtk_src_callbacks.c
  head/multimedia/handbrake/files/patch-libhb_fifo.c
  head/multimedia/handbrake/files/patch-libhb_module.defs
  head/multimedia/handbrake/files/patch-libhb_nlmeans.c
  head/multimedia/handbrake/files/patch-libhb_nlmeans__x86.c
  head/multimedia/handbrake/files/patch-make_configure.py
  head/multimedia/handbrake/files/patch-make_include_main.defs
  head/multimedia/handbrake/files/patch-scripts_build-presets.sh
  head/multimedia/handbrake/files/patch-scripts_create__resources.py
  head/multimedia/handbrake/files/patch-scripts_mac-toolchain-build
  head/multimedia/handbrake/files/patch-scripts_manicure.rb
  head/multimedia/handbrake/files/patch-scripts_mingw-w64-build
  head/multimedia/handbrake/files/patch-scripts_quotestring.py
  head/multimedia/handbrake/files/patch-scripts_repo-info.sh
  head/multimedia/handbrake/files/patch-scripts_tag-release.sh
  head/multimedia/handbrake/files/patch-test_module.defs
  head/multimedia/handbrake/files/patch-version.txt
Comment 5 Fernando Apesteguía freebsd_committer freebsd_triage 2018-04-18 21:34:10 UTC
Committed, thanks!