Bug 207779 - [exp-run] print/harfbuzz split on ICU and non-ICU parts
Summary: [exp-run] print/harfbuzz split on ICU and non-ICU parts
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Koop Mast
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-03-07 14:29 UTC by Vladimir Kondratyev
Modified: 2017-04-27 21:38 UTC (History)
2 users (show)

See Also:
kwm: exp-run?

Attachments
split-harfbuzz.patch (7.43 KB, patch)
2016-03-07 14:29 UTC, Vladimir Kondratyev 		no flags Details | Diff
harfbuzz-icu poudriere log (30.91 KB, text/plain)
2016-03-07 14:31 UTC, Vladimir Kondratyev 		no flags Details
Reworked harfbuzz-icu split off patch (9.82 KB, patch)
2017-04-21 18:35 UTC, Koop Mast 		no flags Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.