Bug 207779

Summary: [exp-run] print/harfbuzz split on ICU and non-ICU parts
Product: Ports & Packages Reporter: Vladimir Kondratyev <wulf>
Component: Individual Port(s)Assignee: Koop Mast <kwm>
Status: Closed FIXED    
Severity: Affects Only Me CC: kwm, portmgr
Priority: --- Keywords: patch
Version: LatestFlags: kwm: exp-run?
Hardware: Any   
OS: Any   
Attachments:
Description Flags
split-harfbuzz.patch
none
harfbuzz-icu poudriere log
none
Reworked harfbuzz-icu split off patch none

Description Vladimir Kondratyev freebsd_committer freebsd_triage 2016-03-07 14:29:26 UTC
Created attachment 167815 [details]
split-harfbuzz.patch

Avoid installation of MONTSTEROUS ICU library with light-weight DMs like XFCE as it unconditionaly sucked in to system via pango->harfbuzz->icu dependency chain.
This patch strip off ICU-dependent part of harfbuzz into separate port named hardbuzz-icu. Grepping shows that the only users of harfbuzz icu backend are libreoffice, chromium and webkit-gtk, so resulting patch is not very big.
Comment 1 Vladimir Kondratyev freebsd_committer freebsd_triage 2016-03-07 14:31:56 UTC
Created attachment 167816 [details]
harfbuzz-icu poudriere log
Comment 2 Koop Mast freebsd_committer freebsd_triage 2017-04-21 18:35:35 UTC
Created attachment 181982 [details]
Reworked harfbuzz-icu split off patch
Comment 3 Koop Mast freebsd_committer freebsd_triage 2017-04-21 18:39:03 UTC
Assign over to portmgr@ for exp-run.

Please run a exp-run for this patch. I have checked all the ports that define a dependency on harfbuzz. Just want to be sure to catch any "hidden" harfbuzz-icu users.
Comment 4 Antoine Brodin freebsd_committer freebsd_triage 2017-04-26 05:28:53 UTC
Exp-run looks fine (done on ports/head@439347)
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-04-27 21:32:14 UTC
A commit references this bug:

Author: kwm
Date: Thu Apr 27 21:31:11 UTC 2017
New revision: 439575
URL: https://svnweb.freebsd.org/changeset/ports/439575

Log:
  Split off the harfbuzz icu support into it's own port.

  PR:		207779
  Submitted by:	Vladimir Kondratyev <vladimir@kondratyev.su>
  Exp-run by:	antoine@

Changes:
  head/editors/libreoffice/Makefile
  head/print/Makefile
  head/print/harfbuzz/Makefile
  head/print/harfbuzz/pkg-plist
  head/print/harfbuzz-icu/
  head/print/harfbuzz-icu/Makefile
  head/print/harfbuzz-icu/pkg-plist
  head/print/tex-xetex/Makefile
  head/print/texlive-base/Makefile
  head/www/chromium/Makefile
  head/www/webkit-gtk2/Makefile
  head/www/webkit-gtk3/Makefile
  head/www/webkit2-gtk3/Makefile
Comment 6 Koop Mast freebsd_committer freebsd_triage 2017-04-27 21:38:28 UTC
Committed thanks.