Filippo Giunchedi has submitted this change and it was merged. Change subject: Add fontconfig file for the pdf render service ......................................................................
Add fontconfig file for the pdf render service Enable font hinting by installing the default electron-render-service .fonts.conf [1] in the service user's home directory. Alternatively, we could mess with the global configuration, but this seemed to be the simpler approach, and is confirmed to work in labs. [1]: https://github.com/msokk/electron-render-service/blob/master/.fonts.conf Change-Id: I35789b89d50260ca79aa97576836f4b35463a776 --- A modules/pdfrender/files/fonts.conf M modules/pdfrender/manifests/init.pp 2 files changed, 56 insertions(+), 0 deletions(-) Approvals: Filippo Giunchedi: Looks good to me, approved Mobrovac: Looks good to me, but someone else must approve jenkins-bot: Verified diff --git a/modules/pdfrender/files/fonts.conf b/modules/pdfrender/files/fonts.conf new file mode 100644 index 0000000..fdc875b --- /dev/null +++ b/modules/pdfrender/files/fonts.conf @@ -0,0 +1,29 @@ +<?xml version='1.0'?> +<!DOCTYPE fontconfig SYSTEM 'fonts.dtd'> +<fontconfig> + <match target="font"> + <edit mode="assign" name="rgba"> + <const>rgb</const> + </edit> + </match> + <match target="font"> + <edit mode="assign" name="hinting"> + <bool>true</bool> + </edit> + </match> + <match target="font"> + <edit mode="assign" name="hintstyle"> + <const>hintslight</const> + </edit> + </match> + <match target="font"> + <edit mode="assign" name="antialias"> + <bool>true</bool> + </edit> + </match> + <match target="font"> + <edit mode="assign" name="lcdfilter"> + <const>lcddefault</const> + </edit> + </match> +</fontconfig> diff --git a/modules/pdfrender/manifests/init.pp b/modules/pdfrender/manifests/init.pp index a1a48ca..58dbe31 100644 --- a/modules/pdfrender/manifests/init.pp +++ b/modules/pdfrender/manifests/init.pp @@ -100,6 +100,33 @@ before => Base::Service_unit['pdfrender'], } + # Enable font hinting + file { "${home_dir}/.config": + ensure => directory, + owner => 'pdfrender', + group => 'pdfrender', + mode => '0555', + } + + file { "${home_dir}/.config/fontconfig": + ensure => directory, + owner => 'pdfrender', + group => 'pdfrender', + mode => '0555', + require => File["${home_dir}/.config"], + before => File["${home_dir}/.config/fontconfig/fonts.conf"], + } + + file { "${home_dir}/.config/fontconfig/fonts.conf": + ensure => present, + owner => 'pdfrender', + group => 'pdfrender', + mode => '0444', + source => 'puppet:///modules/pdfrender/fonts.conf', + before => Base::Service_unit['pdfrender'], + } + # end font hinting + systemd::syslog { 'pdfrender': readable_by => 'all', base_dir => $::service::configuration::log_dir, -- To view, visit https://gerrit.wikimedia.org/r/324747 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I35789b89d50260ca79aa97576836f4b35463a776 Gerrit-PatchSet: 6 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: GWicke <gwi...@wikimedia.org> Gerrit-Reviewer: Faidon Liambotis <fai...@wikimedia.org> Gerrit-Reviewer: Filippo Giunchedi <fgiunch...@wikimedia.org> Gerrit-Reviewer: GWicke <gwi...@wikimedia.org> Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org> Gerrit-Reviewer: Mobrovac <mobro...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits